V592 The expression was enclosed by parentheses twice: ((expression)). One pair of parentheses is unnecessary or misprint is present. light.h 102 V592 The expression was enclosed by parentheses twice: ((expression)). One pair of parentheses is unnecessary or misprint is present. light.h 113 V659 Declarations of functions with 'binary_search' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 409, 422. irrarray.h 422 V659 Declarations of functions with 'getTextureMatrix' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 122, 134. smateriallayer.h 134 V636 The 'p.Y / - 333' expression was implicitly casted from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. biome.cpp 242 V519 The 'ret' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 206, 208. iq3shader.h 208 V595 The 'puncher' pointer was utilized before it was verified against nullptr. Check lines: 1280, 1294. content_sao.cpp 1280 V547 Expression is always false. Unsigned type value is never < 0. environment.cpp 2387 V668 There is no sense in testing the 'src' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. game.cpp 1741 V636 The 'event.player_damage.amount / 2' expression was implicitly casted from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. game.cpp 2274 V581 The conditional expressions of the 'if' operators situated alongside each other are identical. Check lines: 3233, 3244. game.cpp 3244 V581 The conditional expressions of the 'if' operators situated alongside each other are identical. Check lines: 3452, 3458. game.cpp 3458 V668 There is no sense in testing the 'gui_chat_console' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. game.cpp 3499 V668 There is no sense in testing the 'sky' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. game.cpp 3501 V537 Consider reviewing the correctness of 'X' item's usage. guiformspecmenu.cpp 1683 V537 Consider reviewing the correctness of 'X' item's usage. guiformspecmenu.cpp 1731 V592 The expression was enclosed by parentheses twice: ((expression)). One pair of parentheses is unnecessary or misprint is present. guipausemenu.cpp 118 V636 The 'rect.getHeight() / 16' expression was implicitly casted from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. hud.cpp 422 V636 The 'rect.getWidth() / 16' expression was implicitly casted from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. hud.cpp 423 V636 The 'rect.getHeight() / 16' expression was implicitly casted from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. hud.cpp 424 V519 The 'found' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 755, 757. inventorymanager.cpp 757 V547 Expression 'item.wear > 65535' is always false. The value range of unsigned short type: [0, 65535]. l_item.cpp 116 V547 Expression 'bytes_read < 0' is always false. Unsigned type value is never < 0. l_mainmenu.cpp 830 V593 Consider reviewing the expression of the 'A = B != C' kind. The expression is calculated as following: 'A = (B != C)'. l_mainmenu.cpp 831 V595 The 'm_mesh' pointer was utilized before it was verified against nullptr. Check lines: 1260, 1273. mapblock_mesh.cpp 1260 V519 The 'invert' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 198, 202. mapgen_math.cpp 202 V501 There are identical sub-expressions to the left and to the right of the '-' operator: blockpos_max.Z - blockpos_max.Z mapgen_v6.cpp 375 V625 Consider inspecting the 'for' operator. Initial and final values of the iterator are the same. nodedef.cpp 506 V590 Consider inspecting this expression. The expression is excessive or contains a misprint. pathfinder.cpp 524 V547 Expression 'deployed < 0' is always false. Unsigned type value is never < 0. server.cpp 1883 V668 There is no sense in testing the 'inv' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. server.cpp 5177 V595 The 'log' pointer was utilized before it was verified against nullptr. Check lines: 5213, 5232. server.cpp 5213 V640 The code's operational logic does not correspond with its formatting. The statement is indented to the right, but it is always executed. It is possible that curly brackets are missing. treegen.cpp 651