cstylecasts and suppressions

Merge pull request #288 from isildur-g/bugfix-assert

revert a change to assert

Merge pull request #287 from gtsoul-tech/bugFix/cppcheck-cStylecasts-Part4

Part 4 of C-style cast cppcheck

Merge pull request #283 from isildur-g/wip-cppcheck271-part2

Wip cppcheck271 useStlAlgorithm part2

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.

cstylecasts suppress,fixes

undo that one, it breaks

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

Fix remaining C style casts and suppressions

fix cStyleCasts

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

stl'ed another.

stl'ed another one

stl'ed one more instance

cStyleCasts

cStyleCasts

clean up comments

Merge branch 'develop' into wip-cppcheck271-part2

last batch

next batch

next batch

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

Part 1 of C-style cast cppcheck fixes

first batch of cppcheck disables and a few more stl-ifications,
involving use of accumulate() .

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