Erreurs/Warnings Sources Game Files 2016 Team FE V3

[Eristoff 8]

Bonjour,

 

je souhaiterais résoudre les erreurs/warnings des sources game, Files 2016 V3 de la Team FE.

Les résoudre petit à petit.

 

Voici ce que j'ai:

acce.cpp: In function 'void FN_remove_material(CItem**, DWORD, int, int)':
acce.cpp:87: warning: unused variable 'count'
acce.cpp: In function 'bool Acce_InformationInitialize()':
acce.cpp:943: warning: comparison between signed and unsigned integer expression                                                                                        s
char.cpp:7355: warning: this decimal constant is unsigned only in ISO C90
cmd_gm.cpp:2155:9: warning: multi-character character constant
PetSystem.cpp: In member function 'virtual bool CPetActor::_UpdateFollowAI()':
PetSystem.cpp:247: warning: unused variable 'bDoMoveAlone'
PetSystem.cpp: In member function 'CPetActor* CPetSystem::Summon(DWORD, CItem*,                                                                                         const char*, bool, DWORD)':
PetSystem.cpp:546: warning: unused variable 'petVID'
char_item.cpp: In member function 'bool CHARACTER::UseItemEx(CItem*, TItemPos)':
char_item.cpp:1792: warning: format '%lu' expects type 'long unsigned int', but                                                                                         argument 4 has type 'DWORD'
char_item.cpp:1792: warning: format '%lu' expects type 'long unsigned int', but                                                                                         argument 4 has type 'DWORD'
char_item.cpp:1819: warning: suggest parentheses around assignment used as truth                                                                                         value
char.cpp: In member function 'DWORD CHARACTER::PetGetNextExp() const':
char.cpp:7359: warning: control reaches end of non-void function
cmd_general.cpp: In function 'void do_CubePetAdd(CHARACTER*, const char*, int, i                                                                                        nt)':
cmd_general.cpp:2334: warning: unused variable 'strArg1'
char_item.cpp: In member function 'void CHARACTER::BuffOnAttr_ValueChange(BYTE,                                                                                         BYTE, BYTE)':
char_item.cpp:6637: warning: 'pBuff' may be used uninitialized in this function
input_main.cpp: In member function 'void CInputMain::BraveRequestPetName(CHARACT                                                                                        ER*, const char*)':
input_main.cpp:675: warning: format '%lu' expects type 'long unsigned int', but                                                                                         argument 4 has type 'DWORD'
input_main.cpp:675: warning: format '%lu' expects type 'long unsigned int', but                                                                                         argument 4 has type 'DWORD'
questlua_npc.cpp: In function 'int quest::npc_get_vid_attack_mul(lua_State*)':
questlua_npc.cpp:293: warning: unused variable 'q'
questlua_npc.cpp: In function 'int quest::npc_set_vid_attack_mul(lua_State*)':
questlua_npc.cpp:309: warning: unused variable 'q'
questlua_npc.cpp: In function 'int quest::npc_get_vid_damage_mul(lua_State*)':
questlua_npc.cpp:324: warning: unused variable 'q'
questlua_npc.cpp: In function 'int quest::npc_set_vid_damage_mul(lua_State*)':
questlua_npc.cpp:340: warning: unused variable 'q'
../../../Extern/include/cryptopp/algparam.h: In constructor 'CryptoPP::ConstByteArrayParameter::ConstByteArrayParameter(const T&, bool) [with T = std::basic_string, std::allocator >]':
../../../Extern/include/cryptopp/filters.h:793:   instantiated from here
../../../Extern/include/cryptopp/algparam.h:26: warning: unused variable 'cryptopp_assert_26'
compile questlua_dragonsoul.cpp
buff_on_attributes.cpp: In member function 'void CBuffOnAttributes::ChangeBuffValue(BYTE)':
buff_on_attributes.cpp:113: warning: unused variable 'old_value'
buff_on_attributes.cpp:114: warning: unused variable 'new_value'
../../../Extern/include/cryptopp/algparam.h: In member function 'void CryptoPP::AlgorithmParametersTemplate::MoveInto(void*) const [with T = CryptoPP::RandomNumberGenerator*]':
cipher.cpp:397:   instantiated from here
../../../Extern/include/cryptopp/algparam.h:322: warning: unused variable 'p'
group_text_parse_tree.h: In constructor 'CGroupTextParseTreeLoader::CGroupTextParseTreeLoader()':
group_text_parse_tree.h:93: warning: 'CGroupTextParseTreeLoader::m_dwcurLineIndex' will be initialized after
group_text_parse_tree.h:91: warning:   'CGroupNode* CGroupTextParseTreeLoader::m_pRootGroupNode'
group_text_parse_tree.cpp:5: warning:   when initialized here
DragonSoul.cpp: In member function 'bool DSManager::ExtractDragonHeart(CHARACTER*, CItem*, CItem*)':
DragonSoul.cpp:350: warning: unused variable 'sum'
DragonSoul.cpp: In member function 'bool DSManager::PullOut(CHARACTER*, TItemPos, CItem*&, CItem*)':
DragonSoul.cpp:435: warning: unused variable 'dwVnum'
DragonSoul.cpp: In member function 'bool DSManager::DoRefineGrade(CHARACTER*, TItemPos (&)[15])':
DragonSoul.cpp:555: warning: unused variable 'prob_sum'
DragonSoul.cpp: In member function 'bool DSManager::DoRefineStep(CHARACTER*, TItemPos (&)[15])':
DragonSoul.cpp:763: warning: unused variable 'sum'
New_PetSystem.cpp: In member function 'void CNewPetActor::SetExp(DWORD, int)':
New_PetSystem.cpp:326: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:326: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:335: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:335: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:345: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:346: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:366: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:369: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:372: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:378: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:379: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp:390: warning: comparison between signed and unsigned integer expressions
New_PetSystem.cpp: In member function 'DWORD CNewPetActor::Summon(const char*, CItem*, bool)':
New_PetSystem.cpp:534: warning: format '%lu' expects type 'long unsigned int', but argument 4 has type 'DWORD'
New_PetSystem.cpp:534: warning: format '%lu' expects type 'long unsigned int', but argument 4 has type 'DWORD'
New_PetSystem.cpp: In member function 'virtual bool CNewPetActor::_UpdateFollowAI()':
New_PetSystem.cpp:650: warning: unused variable 'bDoMoveAlone'
New_PetSystem.cpp: In member function 'CNewPetActor* CNewPetSystem::Summon(DWORD, CItem*, const char*, bool, DWORD)':
New_PetSystem.cpp:1251: warning: unused variable 'petVID'
item_manager_read_tables.cpp: In member function 'bool ITEM_MANAGER::ReadMonsterDropItemGroup(const char*)':
item_manager_read_tables.cpp:637: warning: unused variable 'pkGroup'
questlua_petnew.cpp: In function 'int quest::newpet_increaseskill(lua_State*)':
questlua_petnew.cpp:135: warning: NULL used in arithmetic
questlua_petnew.cpp: In function 'int quest::newpet_increaseevolution(lua_State*)':
questlua_petnew.cpp:155: warning: NULL used in arithmetic
questlua_petnew.cpp: In function 'int quest::newpet_get_level(lua_State*)':
questlua_petnew.cpp:174: warning: NULL used in arithmetic
questlua_petnew.cpp: In function 'int quest::newpet_get_evo(lua_State*)':
questlua_petnew.cpp:194: warning: NULL used in arithmetic
questlua_petnew.cpp: In function 'int quest::newpet_restore_pet(lua_State*)':
questlua_petnew.cpp:213: warning: format '%lu' expects type 'long unsigned int', but argument 4 has type 'DWORD'
questlua_petnew.cpp:213: warning: format '%lu' expects type 'long unsigned int', but argument 4 has type 'DWORD'
item_manager_read_tables.cpp:627: warning: 'pkGroup' may be used uninitialized in this function
../../../Extern/include/cryptopp/misc.h: At global scope:
../../../Extern/include/cryptopp/misc.h:548: warning: 'std::string CryptoPP::StringNarrow(const wchar_t*, bool)' defined but not used

 

Merci d'avance.

[Asius 5]

Salut,

 

Et encore, tu compile pas avec Visual 2015, il te trouve des warning toutes les 3 lignes mdr

 

La plupart des erreurs sont simples à comprendre, par exemple:

acce.cpp: In function 'void FN_remove_material(CItem**, DWORD, int, int)':

acce.cpp:87: warning: unused variable 'count'

 

La variable 'count' est déclarée, mais inutilisée, tu peux donc l'enlever.

 

Après là c'est simple, mais dans d'autres cas ça va demander un recodage complet de la fonction, pour qu'elle soit conforme aux standards c++11 (ou c++17).

En plus, ce sont des warning, il faut y faire attention, certes, mais la moitié d'entre eux ne sont que dus au nouvelles normes c++.

 

Cordialement,

[Eristoff 8]

Merci d'avoir répondu, Asius.

Du coup comment je procède pour le 'count'?

 

Merci d'avance

[iryzz 211]

La variable 'count' est déclarée, mais inutilisée, tu peux donc l'enlever

:top:

 

tu en as pour 1h avant de tout corriger c'est assez simple suffit de lire ske te renvoie le compilateur si tu sais pas tu go google =)

[Eristoff 8]

Merci iryzz, j'ai pas du bien chercher... Je vais mieux regarder, merci beaucoup !

 

Asius,

 

Dans acce.cpp j'ai ceci:

static void FN_remove_material(LPITEM *items, DWORD item_vnum, int need_count, int result)
{
[TAB]int count = 0;
[TAB]LPITEM item = NULL;

 

Je l'enlève en commentant la ligne comme ceci?

static void FN_remove_material(LPITEM *items, DWORD item_vnum, int need_count, int result)
{
//[TAB]int count = 0;
[TAB]LPITEM item = NULL;

 

 

Edit: En regardant sur google je trouve bien les soucis mais pas comment les résoudre...

 

Merci d'avance.

[Asius 5]

Salut,

 

Pas besoin de commenter, tu peux tout simplement enlever la ligne

 

Et au passage, le C++ se fiche complètement des tabulations et de l'indentation, tout le contraire du Python, donc ne t'embête pas à mettre les [TAB] dans tes bouts de code

[iryzz 211]

Plop,

 

Eristoff 

static void FN_remove_material(LPITEM *items, DWORD item_vnum, int need_count, int result)

{

//[TAB]int count = 0;

[TAB]LPITEM item = NULL;

 

 

[TAB]int count = 0;===> a supprimer 

 

[TAB]LPITEM item = NULL;  ===> NULL = 0 donc mieux vaux mettre 0 même si certain diront le contraire mais sa fait des warning dans les sources  pour rien.(NULL est simplement un define de 0"en gros")

[Eristoff 8]

Bonjour et merci à tous,

 

D'accord, donc si j'ai bien compris...

Je prend l'exemple de ce warnings:

cmd_general.cpp: In function 'void do_CubePetAdd(CHARACTER*, const char*, int, int)':
cmd_general.cpp:2334: warning: unused variable 'strArg1'

 

Dans les sources j'ai ceci:

const std::string& strArg1 = std::string(arg1);

 

Je supprime ou commente la ligne?

Je fais pareille pour tout les autres warnings " unused variable" c'est ça?

 

Merci d'avance.

[iryzz 211]

cmd_general.cpp:2334: warning: unused variable 'strArg1'

Oui c'est une variable qui ne sert a rien donc tu peux la supprimer

[Eristoff 8]

Je supprime la ligne complète?

Je fais pareille pour les autres erreurs de ce genre, c'est à dire je supprime le code complet?

 

Merci d'avance.

[tiitbeet 21]

Bonjour et merci à tous,

 

D'accord, donc si j'ai bien compris...

Je prend l'exemple de ce warnings:

cmd_general.cpp: In function 'void do_CubePetAdd(CHARACTER*, const char*, int, int)':
cmd_general.cpp:2334: warning: unused variable 'strArg1'

 

Dans les sources j'ai ceci:

const std::string& strArg1 = std::string(arg1);

 

Je supprime ou commente la ligne?

Je fais pareille pour tout les autres warnings " unused variable" c'est ça?

 

Merci d'avance.

 

 

 

Je crois qu'il ton assez dit que c'était une fonction non utiliser et que tu pouvais la supprimer... 3 fois tu pose la question 3 fois ils te répondent que tu peux le supprimer 

 

DONC OUI TU PEUX SUPPRIMER CETTE FONCTION SI TU AS PEUR COMMENTE LA ET VOILA ^^

[Eristoff 8]

Bonjour,

Oui tiitbeet mais si je demande confirmation pour les autres c'est car je l'ai déjà fais et au lieu que les warnings diminuent, ils augmentent.

 

Merci d'avance.

[Eristoff 8]

Petit up,

Quelqu'un aurait réussi à tout résoudre et pourrais m'aider ou me donner un petit coup de main?

 

Merci d'avance.

[Asius 5]

Salut,

 

Donne moi les warning sur lesquels tu bloque encore.

 

COrdialement,

Asius

[Eristoff 8]

Bonjour,

 

Ceci par exemple:

PetSystem.cpp: In member function 'CPetActor* CPetSystem::Summon(DWORD, CItem*, const char*, bool, DWORD)':
PetSystem.cpp:546: warning: unused variable 'petVID'

 

Quand je retire la variable en supprimant la ligne, le game ce compile plus du tout.

 

Merci d'avance.

[Calypso 6807]

Car peut être que la variable est initialisé ?

Il faut nous donner toute la fonction CPetActor::Summon

[Eristoff 8]

Bonjour et merci d'avoir répondu,

 

Voici la fonction:

CPetActor* CPetSystem::Summon(DWORD mobVnum, LPITEM pSummonItem, const char* petName, bool bSpawnFar, DWORD options)
{
CPetActor* petActor = this->GetByVnum(mobVnum);

// µî·ÏµÈ ÆêÀÌ ¾Æ´Ï¶ó¸é »õ·Î »ý¼º ÈÄ °ü¸® ¸ñ·Ï¿¡ µî·ÏÇÔ.
if (0 == petActor)
{
	petActor = M2_NEW CPetActor(m_pkOwner, mobVnum, options);
	m_petActorMap.insert(std::make_pair(mobVnum, petActor));
}

DWORD petVID = petActor->Summon(petName, pSummonItem, bSpawnFar);

if (NULL == m_pkPetSystemUpdateEvent)
{
	petsystem_event_info* info = AllocEventInfo();

	info->pPetSystem = this;

	m_pkPetSystemUpdateEvent = event_create(petsystem_update_event, info, PASSES_PER_SEC(1) / 4);	// 0.25ÃÊ	
}

return petActor;
}

 

Merci d'avance.