Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve: move all onKill events to registed onDeath #1759

Merged
merged 4 commits into from
Nov 2, 2023

Merge branch 'main' into luan/no-more-onkill

98afddc
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

improve: move all onKill events to registed onDeath #1759

Merge branch 'main' into luan/no-more-onkill
98afddc
Select commit
Loading
Failed to load commit list.
GitHub Actions / cppcheck completed Nov 2, 2023 in 0s

reviewdog [cppcheck] report

reported by reviewdog 🐶

Findings (1)

src/lua/functions/creatures/monster/monster_type_functions.cpp|1008| Unused variable: _

Filtered Findings (157)

src/canary_server.cpp|| Too many #ifdef configurations - cppcheck only checks 12 of 24 configurations. Use --force to check all configurations.
src/creatures/creatures_definitions.hpp|1661| Variable 'itemName' is assigned in constructor body. Consider performing initialization in initialization list.
src/security/rsa.hpp|16| Class 'RSA' has a constructor with 1 argument that is not explicit.
src/canary_server.cpp|313| Same expression on both sides of '||' because 'datapackName!="data-canary"&&datapackName!="data-otservbr-global"' and 'datapackName!="data-otservbr-global"&&datapackName!="data-canary"' represent the same value.
src/creatures/creatures_definitions.hpp|1545| Function parameter 'initName' should be passed by const reference.
src/creatures/creatures_definitions.hpp|1545| Function parameter 'initType' should be passed by const reference.
src/game/game.hpp|361| Function parameter 'itemIds' should be passed by const reference.
src/game/scheduling/task.hpp|20| Function parameter 'context' should be passed by const reference.
src/game/scheduling/task.hpp|25| Function parameter 'context' should be passed by const reference.
src/game/scheduling/dispatcher.hpp|101| Function parameter 'context' should be passed by const reference.
src/game/scheduling/dispatcher.hpp|106| Function parameter 'context' should be passed by const reference.
src/server/network/webhook/webhook.hpp|33| Function parameter 'payload' should be passed by const reference.
src/server/network/webhook/webhook.hpp|34| Function parameter 'title' should be passed by const reference.
src/server/network/webhook/webhook.hpp|34| Function parameter 'message' should be passed by const reference.
src/server/network/webhook/webhook.hpp|46| Function parameter 'title' should be passed by const reference.
src/server/network/webhook/webhook.hpp|46| Function parameter 'message' should be passed by const reference.
src/game/zones/zone.hpp|206| Unused variable: _
src/canary_server.cpp|194| Skipping configuration 'LUAJIT_VERSION' since the value of 'LUAJIT_VERSION' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly.
src/canary_server.cpp|247| Skipping configuration 'NDEBUG;_MSC_VER;_WIN32' since the value of '_MSC_VER' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly.
src/canary_server.cpp|247| Skipping configuration '_MSC_VER' since the value of '_MSC_VER' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly.
src/canary_server.cpp|247| Skipping configuration '_MSC_VER;_WIN32' since the value of '_MSC_VER' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly.
src/creatures/appearance/outfit/outfit.cpp|72| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/combat/combat.cpp|1463| Identical inner 'if' condition is always true.
src/creatures/combat/combat.cpp|34| Variable 'magicLevelSkill' is assigned a value that is never used.
src/creatures/combat/combat.cpp|1353| Variable 'pos' is not assigned a value.
src/creatures/combat/condition.cpp|1733| Consider using std::accumulate algorithm instead of a raw loop.
src/creatures/creature.cpp|1364| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/creature.cpp|1373| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/creature.cpp|1513| Consider using std::any_of algorithm instead of a raw loop.
src/creatures/interactions/chat.cpp|480| The statement 'if (type!=TALKTYPE_CHANNEL_Y) type=TALKTYPE_CHANNEL_Y' is logically equivalent to 'type=TALKTYPE_CHANNEL_Y'.
src/creatures/monsters/monster.cpp|879| Consider using std::any_of algorithm instead of a raw loop.
src/creatures/monsters/monster.cpp|1010| Consider using std::count_if algorithm instead of a raw loop.
src/creatures/monsters/monster.cpp|1079| Variable 'x' is not assigned a value.
src/creatures/monsters/monsters.cpp|257| Condition '!combatSpell' is always false
src/creatures/monsters/monsters.cpp|265| Condition 'combatSpell' is always true
src/creatures/monsters/spawns/spawn_monster.cpp|273| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/npcs/npc.cpp|267| Consider using std::accumulate algorithm instead of a raw loop.
src/creatures/npcs/npc.cpp|376| Consider using std::accumulate algorithm instead of a raw loop.
src/creatures/npcs/npc.cpp|660| Unused variable: _
src/creatures/players/grouping/familiars.cpp|66| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/grouping/groups.cpp|104| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/grouping/guild.cpp|37| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/grouping/guild.cpp|46| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/grouping/guild.cpp|55| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/grouping/party.cpp|476| Variable 'highestLevel' is assigned a value that is never used.
src/creatures/players/grouping/party.cpp|751| Variable '(*it).second' is assigned a value that is never used.
src/creatures/players/grouping/party.cpp|778| Variable '(*it).second' is assigned a value that is never used.
src/creatures/players/imbuements/imbuements.cpp|342| Unused variable: key
src/creatures/players/player.cpp|366| Redundant condition: shield. '!A || (A && B)' is equivalent to '!A || B'
src/creatures/players/player.cpp|395| Condition 'weapon' is always false
src/creatures/players/player.cpp|400| Condition 'shield' is always false
src/creatures/players/player.cpp|401| Condition 'weapon!=nullptr' is always false
src/creatures/players/player.cpp|2723| The scope of the variable 'maxBlessing' can be reduced.
src/creatures/players/player.cpp|7682| The scope of the variable 'stage' can be reduced.
src/creatures/players/player.cpp|836| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/player.cpp|1861| Consider using std::any_of algorithm instead of a raw loop.
src/creatures/players/player.cpp|4595| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/player.cpp|4836| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/player.cpp|4857| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/player.cpp|4911| Consider using std::any_of algorithm instead of a raw loop.
src/creatures/players/player.cpp|4920| Consider using std::any_of algorithm instead of a raw loop.
src/creatures/players/player.cpp|7609| Consider using std::accumulate algorithm instead of a raw loop.
src/creatures/players/player.cpp|7668| Consider using std::accumulate algorithm instead of a raw loop.
src/creatures/players/player.cpp|625| Unused variable: key
src/creatures/players/player.cpp|2852| Unused variable: key
src/creatures/players/player.cpp|2906| Unused variable: key
src/creatures/players/player.cpp|2912| Unused variable: key
src/creatures/players/player.cpp|6978| Variable 'mapTier' is not assigned a value.
src/creatures/players/player.cpp|7051| Variable 'mapTier' is not assigned a value.
src/creatures/players/player.cpp|7174| Variable 'mapTier' is not assigned a value.
src/creatures/players/player.cpp|7622| Variable 'stage' is assigned a value that is never used.
src/creatures/players/vocations/vocation.cpp|194| Consider using std::find_if algorithm instead of a raw loop.
src/creatures/players/wheel/player_wheel.cpp|1687| The scope of the variable 'magicSkill' can be reduced.
src/creatures/players/wheel/player_wheel.cpp|1688| The scope of the variable 'distanceSkill' can be reduced.
src/creatures/players/wheel/player_wheel.cpp|1716| The scope of the variable 'newCritical' can be reduced.
src/creatures/players/wheel/player_wheel.cpp|74| struct member 'PromotionScroll::itemId' is never used.
src/creatures/players/wheel/player_wheel.cpp|76| struct member 'PromotionScroll::extraPoints' is never used.
src/creatures/players/wheel/player_wheel.cpp|2479| Variable 'name_it' is not assigned a value.
src/game/scheduling/events_scheduler.cpp|90| Local variable 'eventName' shadows outer variable
src/game/scheduling/save_manager.cpp|19| Unused variable: _
src/game/scheduling/save_manager.cpp|25| Unused variable: _
src/game/zones/zone.cpp|97| Consider using std::copy algorithm instead of a raw loop.
src/game/zones/zone.cpp|141| Unused variable: _
src/game/zones/zone.cpp|149| Unused variable: _
src/game/zones/zone.cpp|157| Unused variable: _
src/game/zones/zone.cpp|172| Unused variable: _
src/io/functions/iologindata_load_player.cpp|838| Unused variable: id
src/io/functions/iologindata_save_player.cpp|38| Condition '!container' is always false
src/io/functions/iologindata_save_player.cpp|329| Variable 'itemId' is not assigned a value.
src/io/functions/iologindata_save_player.cpp|781| Variable 'key' is not assigned a value.
src/io/io_bosstiary.cpp|289| Consider using std::count_if algorithm instead of a raw loop.
src/io/io_bosstiary.cpp|76| Variable 'randomBossId' is not assigned a value.
src/io/io_bosstiary.cpp|156| Variable 'bossRaceId' is not assigned a value.
src/io/iobestiary.cpp|100| Consider using std::find_if algorithm instead of a raw loop.
src/io/iomap.cpp|125| The scope of the variable 'tileIsStatic' can be reduced.
src/io/iomapserialize.cpp|78| Unused variable: key
src/io/iomapserialize.cpp|326| Unused variable: key
src/io/iomapserialize.cpp|342| Unused variable: key
src/items/functions/item/attribute.cpp|50| Consider using std::find_if algorithm instead of a raw loop.
src/items/functions/item/item_parse.cpp|716| Condition 'conditionDamage' is always false
src/items/functions/item/item_parse.cpp|551| Consider using std::transform algorithm instead of a raw loop.
src/items/item.cpp|2843| Condition 'subType!=1' is always true
src/items/item.cpp|1013| The scope of the variable 'isTradeable' can be reduced.
src/items/tile.cpp|1791| Condition 'items' is always true
src/kv/value_wrapper.cpp|39| Unused variable: _
src/lua/creature/creatureevent.cpp|17| Unused variable: name
src/lua/creature/movement.cpp|135| Consider using std::any_of algorithm instead of a raw loop.
src/lua/functions/core/game/game_functions.cpp|41| Condition '!monsterType' is always false
src/lua/functions/core/game/game_functions.cpp|66| Local variable 'alternateName' shadows outer variable
src/lua/functions/creatures/combat/spell_functions.cpp|617| Either the condition 'vocList.size()>0' is redundant or vocList size can be 1. Expression 'vocList[1]' cause access out of bounds.
src/lua/functions/creatures/creature_functions.cpp|508| Expression is always true because 'else if' condition is opposite to previous condition at line 506.
src/lua/functions/creatures/creature_functions.cpp|784| Condition 'oldPosition.x<position.x' is always true
src/lua/functions/creatures/npc/npc_functions.cpp|502| The scope of the variable 'shoppingBagSlots' can be reduced.
src/lua/functions/creatures/npc/npc_functions.cpp|521| Consider using std::find_if algorithm instead of a raw loop.
src/lua/functions/creatures/npc/npc_functions.cpp|527| Unused variable: _
src/lua/functions/creatures/player/player_functions.cpp|3052| Unused variable: key
src/lua/functions/items/item_functions.cpp|721| Condition 'moveItem' is always false
src/lua/scripts/script_environment.cpp|77| Consider using std::find_if algorithm instead of a raw loop.
src/main.cpp|| Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations.
src/map/house/house.cpp|528| Local variable 'line' shadows outer variable
src/map/house/house.cpp|690| Consider using std::find_if algorithm instead of a raw loop.
src/protobuf/appearances.pb.cc|| This file is not analyzed. Cppcheck failed to extract a valid configuration. Use -v for more details.
src/protobuf/kv.pb.cc|| This file is not analyzed. Cppcheck failed to extract a valid configuration. Use -v for more details.
src/security/rsa.cpp|164| The scope of the variable 'buffer' can be reduced.
src/server/network/protocol/protocolgame.cpp|345| Condition 'containerType==0' is always true
src/server/network/protocol/protocolgame.cpp|3039| Condition 'counter>0' is always true
src/server/network/protocol/protocolgame.cpp|57| Consider using std::distance algorithm instead of a raw loop.
src/server/network/protocol/protocolgame.cpp|349| Consider using std::accumulate algorithm instead of a raw loop.
src/server/network/protocol/protocolgame.cpp|2833| Unused variable: ss
src/server/network/protocol/protocolgame.cpp|3959| Variable 'flag' is assigned a value that is never used.
src/server/network/protocol/protocolgame.cpp|3967| Variable 'flag' is assigned a value that is never used.
src/server/network/protocol/protocolgame.cpp|4958| Variable 'iteratorItemId' is not assigned a value.
src/server/network/protocol/protocolgame.cpp|7031| Variable 'masterPlayer' is assigned a value that is never used.
src/server/network/protocol/protocolgame.cpp|7917| Variable 'itemTier' is not assigned a value.
src/server/network/protocol/protocolstatus.cpp|100| Unused variable: key
src/utils/tools.cpp|1614| Condition 'number>=7400' is always true
src/utils/tools.cpp|484| Function parameter 'string' should be passed by const reference.
src/utils/tools.cpp|193| Consider using std::accumulate algorithm instead of a raw loop.
src/utils/tools.cpp|378| Consider using std::transform algorithm instead of a raw loop.
src/utils/tools.cpp|1027| Consider using std::find_if algorithm instead of a raw loop.
src/utils/tools.cpp|454| The function 'formatTime' is never used.
src/utils/tools.cpp|198| The function 'generateToken' is never used.
src/utils/tools.cpp|1547| The function 'getObjectCategoryName' is never used.
src/utils/tools.cpp|1673| The function 'getPlayerObjectPronoun' is never used.
src/utils/tools.cpp|1690| The function 'getPlayerPossessivePronoun' is never used.
src/utils/tools.cpp|1707| The function 'getPlayerReflexivePronoun' is never used.
src/utils/tools.cpp|1656| The function 'getPlayerSubjectPronoun' is never used.
src/utils/tools.cpp|920| The function 'getSkullType' is never used.
src/utils/tools.cpp|940| The function 'getSpawnType' is never used.
src/utils/tools.cpp|1724| The function 'getVerbForPronoun' is never used.
src/utils/tools.cpp|912| The function 'getWeaponAction' is never used.
src/utils/pugicast.cpp|13| The function 'logError' is never used.
src/utils/tools.cpp|274| The function 'toCamelCase' is never used.
src/utils/tools.cpp|330| The function 'toKebabCase' is never used.
src/utils/tools.cpp|314| The function 'toSnakeCase' is never used.
src/utils/tools.cpp|1514| The function 'validateName' is never used.
nofile|| Cppcheck cannot find all the include files (use --check-config for details)

Annotations

Check warning on line 1008 in src/lua/functions/creatures/monster/monster_type_functions.cpp

See this annotation in the file changed.

@github-actions github-actions / cppcheck

[cppcheck] src/lua/functions/creatures/monster/monster_type_functions.cpp#L1008

Unused variable: _
Raw output
src/lua/functions/creatures/monster/monster_type_functions.cpp:1008:Unused variable: _