Function parameter 'name' should be passed by const reference.

Function parameter 'icon' should be passed by const reference.

Function parameter 'parentName' should be passed by const reference.

Function parameter 'name' should be passed by const reference.

Function parameter 'icon' should be passed by const reference.

Function parameter 'description' should be passed by const reference.

Consider using std::any_of algorithm instead of a raw loop.

Consider using std::find_if algorithm instead of a raw loop.

Consider using std::find_if algorithm instead of a raw loop.

Consider using std::any_of algorithm instead of a raw loop.

Consider using std::any_of algorithm instead of a raw loop.

Variable 'entryPages' is assigned a value that is never used.

The function 'isExerciseWeapon' is never used.

The function 'isKegItem' is never used.

narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int32_t' (aka 'int') is implementation-defined

0xFB is a magic number; consider replacing it with a named constant

0xFC is a magic number; consider replacing it with a named constant

0xFD is a magic number; consider replacing it with a named constant

0xEF is a magic number; consider replacing it with a named constant

0xFA is a magic number; consider replacing it with a named constant

0xFB is a magic number; consider replacing it with a named constant

'auto offer' can be declared as 'const auto *offer'

implicit conversion 'const Offer *' -> 'bool'

26 is a magic number; consider replacing it with a named constant

MISRA 5-0-5: There shall be no implicit floating-integral conversions

MISRA 5-0-5: There shall be no implicit floating-integral conversions

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

backward branch (for loop) is ID-dependent due to variable reference to 'entry' and may cause performance degradation

0xFD is a magic number; consider replacing it with a named constant

'auto currentCategory' can be declared as 'const auto *currentCategory'

implicit conversion 'const Category *' -> 'bool'

'auto offer' can be declared as 'const auto *offer'

'auto parentCategory' can be declared as 'const auto *parentCategory'

implicit conversion 'const Category *' -> 'bool'

implicit conversion 'bool' -> 'uint8_t' (aka 'unsigned char')

0xFC is a magic number; consider replacing it with a named constant

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

loop variable is copied but only used as const reference; consider making it a const reference

'auto offer' can be declared as 'const auto *offer'

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

MISRA 2-13-3: A "U" suffix shall be applied to all octal or hexadecimal integer literals of unsigned type

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

MISRA 2-13-3: A "U" suffix shall be applied to all octal or hexadecimal integer literals of unsigned type

0xFC is a magic number; consider replacing it with a named constant

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

loop variable is copied but only used as const reference; consider making it a const reference

0xFC is a magic number; consider replacing it with a named constant

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

loop variable is copied but only used as const reference; consider making it a const reference

MISRA 2-13-3: A "U" suffix shall be applied to all octal or hexadecimal integer literals of unsigned type

0xEA is a magic number; consider replacing it with a named constant

'auto currentOffer' can be declared as 'const auto *currentOffer'

redundant string initialization

variable 'sexId' is not initialized

the parameter 'successMessage' is copied for each invocation but only used as a const reference; consider making it a const reference

the parameter 'errorMessage' is copied for each invocation but only used as a const reference; consider making it a const reference

MISRA 2-13-3: A "U" suffix shall be applied to all octal or hexadecimal integer literals of unsigned type

0xE0 is a magic number; consider replacing it with a named constant

2 adjacent parameters of 'requestPurchaseData' of convertible types are easily swapped by mistake

function 'canBuyStoreOffer' has cognitive complexity of 34 (threshold 25)

962 is a magic number; consider replacing it with a named constant

975 is a magic number; consider replacing it with a named constant

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

50 is a magic number; consider replacing it with a named constant

8 is a magic number; consider replacing it with a named constant

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

5 is a magic number; consider replacing it with a named constant

8 is a magic number; consider replacing it with a named constant

5 is a magic number; consider replacing it with a named constant

14901 is a magic number; consider replacing it with a named constant

90 is a magic number; consider replacing it with a named constant

6 is a magic number; consider replacing it with a named constant

function 'checkMoveItemToCylinder' has cognitive complexity of 82 (threshold 25)

the parameter 'receptorName' is copied for each invocation but only used as a const reference; consider making it a const reference

MISRA 5-0-12: signed char and unsigned char type shall only be used for the storage and use of numeric values

MISRA 5-0-12: signed char and unsigned char type shall only be used for the storage and use of numeric values

narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int32_t' (aka 'int') is implementation-defined

narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int32_t' (aka 'int') is implementation-defined

2 adjacent parameters of 'playerOpenStoreHistory' of similar type ('uint32_t') are easily swapped by mistake

function 'playerBuyStoreOffer' has cognitive complexity of 48 (threshold 25)

parameter 'sexId' is unused

implicit conversion 'const Offer *' -> 'bool'

implicit conversion 'bool' -> 'uint16_t' (aka 'unsigned short')

962 is a magic number; consider replacing it with a named constant

975 is a magic number; consider replacing it with a named constant

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

128 is a magic number; consider replacing it with a named constant

136 is a magic number; consider replacing it with a named constant

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

3600 is a magic number; consider replacing it with a named constant

6 is a magic number; consider replacing it with a named constant

8 is a magic number; consider replacing it with a named constant

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

8 is a magic number; consider replacing it with a named constant

65175 is a magic number; consider replacing it with a named constant

narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int32_t' (aka 'int') is implementation-defined

3000 is a magic number; consider replacing it with a named constant

50 is a magic number; consider replacing it with a named constant

14901 is a magic number; consider replacing it with a named constant

90 is a magic number; consider replacing it with a named constant

14901 is a magic number; consider replacing it with a named constant

MISRA 5-0-12: signed char and unsigned char type shall only be used for the storage and use of numeric values

redundant explicit casting to the same type 'uint32_t' (aka 'unsigned int') as the sub-expression, remove this casting

method 'processHouseOffer' can be made static

2 adjacent parameters of 'processHouseOffer' of convertible types are easily swapped by mistake

method 'processChargesOffer' can be made static

2 adjacent parameters of 'processChargesOffer' of convertible types are easily swapped by mistake

MISRA 0-1-7: The value returned by a function having non-void return type that is not an overloaded operator shall always be used

redundant boolean literal in conditional return statement

method 'processStackableOffer' can be made static

MISRA 5-0-6: An implicit integral or floating-point conversion shall not reduce the size of the underlying type

MISRA 0-1-7: The value returned by a function having non-void return type that is not an overloaded operator shall always be used

redundant boolean literal in conditional return statement

repeated branch body in conditional chain

redundant boolean literal in conditional return statement

function 'loadFromXml' has cognitive complexity of 40 (threshold 25)

MISRA 5-3-1: Each operand of the ! operator, the logical && or the logical || operators shall have type bool

variable 'categoryRook' is not initialized

redundant boolean literal in conditional assignment

MISRA 6-4-2: All if ... else if constructs shall be terminated with an else clause

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

MISRA 5-3-1: Each operand of the ! operator, the logical && or the logical || operators shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_node ***)') -> 'bool'

variable 'subCategoryRook' is not initialized

redundant boolean literal in conditional assignment

variable 'subCategoryState' is not initialized

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

MISRA 5-3-1: Each operand of the ! operator, the logical && or the logical || operators shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_node ***)') -> 'bool'

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

redundant string initialization

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_attribute ***)') -> 'bool'

variable 'type' is not initialized

uninitialized record type: 'outfitId'

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_attribute ***)') -> 'bool'

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_attribute ***)') -> 'bool'

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_attribute ***)') -> 'bool'

redundant string initialization

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_attribute ***)') -> 'bool'

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_attribute ***)') -> 'bool'

redundant cast to the same type

redundant explicit casting to the same type 'bool' as the sub-expression, remove this casting

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

MISRA 5-3-1: Each operand of the ! operator, the logical && or the logical || operators shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_node ***)') -> 'bool'

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

MISRA 5-0-13: The condition of an if-statement and the condition of an iteration-statement shall have type bool

MISRA 5-3-1: Each operand of the ! operator, the logical && or the logical || operators shall have type bool

implicit conversion 'unspecified_bool_type' (aka 'void (*)(xml_node ***)') -> 'bool'

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

the parameter 'newCategory' is copied for each invocation but only used as a const reference; consider making it a const reference

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

the parameter 'categoryName' is copied for each invocation but only used as a const reference; consider making it a const reference

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

the parameter 'subCategoryName' is copied for each invocation but only used as a const reference; consider making it a const reference

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

MISRA 0-1-7: The value returned by a function having non-void return type that is not an overloaded operator shall always be used

MISRA 0-1-7: The value returned by a function having non-void return type that is not an overloaded operator shall always be used

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

method 'findCategory' can be made const

the parameter 'categoryName' is copied for each invocation but only used as a const reference; consider making it a const reference

'auto currentCategory' can be declared as 'const auto *currentCategory'

MISRA 5-3-1: Each operand of the ! operator, the logical && or the logical || operators shall have type bool

implicit conversion 'const Category *' -> 'bool'

'auto subCat' can be declared as 'const auto *subCat'

return type 'const std::vector<std::string>' (aka 'const vector<basic_string<char>>') is 'const'-qualified at the top level, which may reduce code readability without improving const correctness

the parameter 'newSubCategory' is copied for each invocation but only used as a const reference; consider making it a const reference

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

the parameter 'newOffer' is copied for each invocation but only used as a const reference; consider making it a const reference

kernel performance could be improved by unrolling this loop with a '#pragma unroll' directive

MISRA 6-4-2: All if ... else if constructs shall be terminated with an else clause

do not use 'else' after 'return'

redundant boolean literal in conditional return statement