revert a change to assert , the original logic might have been
subtely clever (or else totally useless all these years), when we
see which of the two we might delete that assert entirely. for now
put it back as it was.

Merge pull request #286 from VectorCamp/bugfix/cppcheck-fix-wrong-casts

Fix remaining C style casts and suppressions

Fix typo

Add more C style casts fixes and suppressions

remove unneeded suppression

Fix more C-style casts

Fix casts

Fix wrong comparison

Fix wrong casts in pcapscan.cc

Merge pull request #285 from gtsoul-tech/bugFix/cppcheck-cStylecasts-Part3

Part 3 of C-style cast cppcheck

Merge pull request #284 from gtsoul-tech/bigFix/cppcheck-cStylecasts-Part2

Part 2 of C-style cast cppcheck

cStyleCasts

cStyleCasts

Merge pull request #281 from VectorCamp/bugfix/cppcheck-cstyleCasts-part1

Part 1 of C-style cast cppcheck fixes

add alternative macro without C casts to avoid Cppcheck warnings

Fix C style casts in mcsheng_compile.cpp

Merge pull request #280 from isildur-g/bugfix-rose-segfault

the segfault we ran into, included_frag_id should be

Fix wrong cast in aligned_free()

Fix C-style casts

Merge branch 'develop' into bugfix-rose-segfault

the segfault we ran into, included_frag_id should be
included_delay_frag_id.

Merge pull request #279 from VectorCamp/bugfix/cppcheck-unreadVariable-others

Fix marked as done cppcheck warnings unreadVariable & others

Merge pull request #278 from VectorCamp/bugfix/cppcheck-remaining-const-warnings

Fix remaining marked as done const* cppcheck warnings

remove unused variables

another duplicateExpression false positive

Fix unreadVariable warning

Fix missingOverride

Fix unknownMacro

Fix variableScope

False positive knownConditionTrueFalse

False positives duplicateExpression

Fix remaining marked as done const* cppcheck warnings

Merge pull request #277 from isildur-g/wip-cppcheck271

phase 1 of addressing cppcheck useStlAlgorithm warnings for fill and copy operations

Merge pull request #276 from gtsoul-tech/bugFix/cppcheck-noExplicitConstructor-part2

Bug fix/cppcheck no explicit constructor part2

phase 1 of addressing cppcheck useStlAlgorithm warnings,
this set only includes fill and copy operations.

supervector conversion

forgot some conversions SuperVector<32>

explicit constructor Supervector

ppc64el supervector explicit constructor

arm supervector explicit constructor

x86 explicit constructor supervector

Merge pull request #274 from gtsoul-tech/bugFix/cppcheckError-noexplicitConstructor

Bug fix/cppcheck error noexplicitconstructor

most ptr.get() conversion

fixes left_id and suffix_id

more fixes

fix conversions

Merge pull request #267 from VectorCamp/bugfix/cppcheck-rose_build_scatter-uselessAssignmentArg

Fix marked as false positive uselessAssignmentArg cppcheck warning

Merge pull request #268 from VectorCamp/bugfix/cppcheck-constStatement

Fix marked as false positive constStatement cppcheck warnings

Merge pull request #269 from VectorCamp/bugfix/cppcheck-duplicateAssignment

Fix marked as false positive duplicateAssignExpression cppcheck warnings

Merge pull request #270 from VectorCamp/bugfix/cppcheck-redundantAssignment

Fix marked as false positive redundantAssignment cppcheck warnings

Merge pull request #271 from VectorCamp/bugfix/cppcheck-identicalConditionAfterEarlyExit

Fix marked as false positive identicalConditionAfterEarlyExit cppcheck warnings

Merge pull request #272 from VectorCamp/bugfix/cppcheck-truncLongCastAssignment

Fix marked as false positive truncLongCastAssignment cppcheck warnings

Merge pull request #273 from VectorCamp/bugfix/cppcheck-knownConditionTrueFalse

 Fix marked as false positive knownConditionTrueFalse cppcheck warnings

revert supervector

fix

Merge branch 'develop' into bugFix/cppcheckError-noexplicitConstructor

nodes is never empty at this stage, emplace_back() is called just previously

Fix marked as false positive knownConditionTrueFalse cppcheck warnings

std::make_shared<> does not return null, it throws std::bad_alloc.

Fix false positive truncLongCastAssignment warnings

Fix false positive /identicalConditionAfterEarlyExit warnings

Fix false positive redundantAssignment warnings

Fix false positive duplicateAssignExpression warnings

Fix false positive constStatement warnings

replace memcpy with std::copy

Fix false positive uselessAssignmentArg cppcheck warning

(also at the same time some cstyleCast warnings)

Merge pull request #265 from isildur-g/wip-isildur-g-cppcheck66

addressing cppcheck shadowFunction warnings

Merge branch 'develop' into test-noExplicitConstructor

supervector

noExplicitConstructor 1 more

noExplicitConstructor

remove test comment

Merge branch 'develop' into wip-isildur-g-cppcheck66

test comment

one more fixed that had been missed.

Merge pull request #264 from gtsoul-tech/bugFix/cppcheck-constVariablePointer

Cppcheck constVariablePointer error

maybe fix broken merge

just a dummy commit to test something

Merge branch 'develop' into wip-isildur-g-cppcheck66

addressing cppcheck shadowFunction warnings

Merge pull request #263 from gtsoul-tech/bug/cppcheck-61

Cppcheck knownConditionTrueFalse error

constVariablePointer

knownConditionTrueFalse fixes previously fp

Merge pull request #262 from isildur-g/wip-isildur-g-cppcheck-47-48-58

addressing 47,48,58

refactor bool to void setDistinctTops setDistinctRoseTops
setDistinctSuffixTops

knownConditionTrueFalse

the merge got screwed up, this should fix it

Merge branch 'develop' into wip-isildur-g-cppcheck-47-48-58

addressing 47 [constParameterReference],48 [constVariableReference],58
[constVariable]

Merge pull request #239 from ypicchi-arm/feature/add-sheng-unit-tests

Feature/add sheng unit tests

Merge pull request #261 from gtsoul-tech/bug/cppCheck-41

Cppcheck unreadVariable error

Add sheng tests

Signed-off-by: Yoan Picchi <yoan.picchi@arm.com>

unreadVariable

Merge pull request #259 from gtsoul-tech/bug/cppcheckErrors

Bug/cppcheck errors (32,35) WIP

Merge pull request #258 from isildur-g/wip-isildur-g-cppcheck1220

Wip cppcheck1220

new variableScope

fixed const adjustments.

variableScopeFix

variableScope

ome of the consts couldnt be propagated and had to be reverted

missingOverride