c++ Trinity Compile error after adding Gunship

[bengelbaer]

Hello,

i need help with an compile error

6>..\..\..\..\TrinityCore\PlaycurseEmu\src\server\scripts\Northrend\IcecrownCitadel\boss_gunship_battle.cpp(2445): warning C4373: 'npc_gunship_portal::npc_gunship_portalAI::UpdateAI': Virtuelle Funktion überschreibt 'ScriptedAI::UpdateAI'. Vorherige Compilerversionen werden nicht überschrieben, wenn sich die Parameter nur durch const/volatile-Qualifizierer unterscheiden.
6>          C:/Server/TrinityCore/PlaycurseEmu/src/server/game/AI/ScriptedAI\ScriptedCreature.h(147): Siehe Deklaration von 'ScriptedAI::UpdateAI'
6>..\..\..\..\TrinityCore\PlaycurseEmu\src\server\scripts\Northrend\IcecrownCitadel\boss_gunship_battle.cpp(3156): warning C4146: Einem vorzeichenlosen Typ wurde ein unärer Minus-Operator zugewiesen. Das Ergebnis ist weiterhin vorzeichenlos.


15>     Bibliothek "C:/Server/TrinityCoreMLZBuild/src/server/worldserver/Release/worldserver.lib" und Objekt "C:/Server/TrinityCoreMLZBuild/src/server/worldserver/Release/worldserver.exp" werden erstellt.
15>game.lib(Transport.obj) : error LNK2019: Verweis auf nicht aufgelöstes externes Symbol ""public: void __thiscall Transport::UpdatePlayerPositions(void)" (?UpdatePlayerPositions@Transport@@QAEXXZ)" in Funktion ""public: void __thiscall Transport::UpdatePosition(struct MovementInfo *)" (?UpdatePosition@Transport@@QAEXPAUMovementInfo@@@Z)".
15>C:\Server\TrinityCoreMLZBuild\bin\Release\worldserver.exe : fatal error LNK1120: 1 nicht aufgelöste externe Verweise.
16>------ Neues Erstellen gestartet: Projekt: ALL_BUILD, Konfiguration: Release Win32 ------
16>  Building Custom Rule C:/Server/TrinityCore/PlaycurseEmu/CMakeLists.txt
16>  CMake does not need to re-run because C:\Server\TrinityCoreMLZBuild\CMakeFiles\generate.stamp is up-to-date.
16>  Build all projects

is anyone here who can help me?

i have uploaded the core on github. So you can look better for the error at this.

https://github.com/bengelbaer/PlaycurseEmu.git

sorry for my english.

Thx

Bengel

 

[Tommy]

Warning Fixes:

Line 2445 in boss_gunship_battle, remove the 'const' from "UpdateAI". It should look like this:

void UpdateAI(uint32 diff)

Line 3156 in boss_gunship_battle, remove the "-" in front of "itemCount" in "DestroyItemCount". It is quite pointless seeing you're already destroying the item and its non-subtractive count. It should look like this:

hitPlr->DestroyItemCount(itemId, itemCount, true, false);

And the reason for the linker error is because "void UpdatePlayerPositions();" in Transport.h is pretty useless because it doesn't do anything. It isn't setup in Transport.cpp, for example:

void Transport::UpdatePlayerPositions()
{
}

It is being used without having a structure. However, I'm guess you missed that? Whatever happened, right now it doesn't do crap.

 

[bengelbaer]

Warning Fixes:

Line 2445 in boss_gunship_battle, remove the 'const' from "UpdateAI". It should look like this:

void UpdateAI(uint32 diff)

Line 3156 in boss_gunship_battle, remove the "-" in front of "itemCount" in "DestroyItemCount". It is quite pointless seeing you're already destroying the item and its non-subtractive count. It should look like this:

hitPlr->DestroyItemCount(itemId, itemCount, true, false);

And the reason for the linker error is because "void UpdatePlayerPositions();" in Transport.h is pretty useless because it doesn't do anything. It isn't setup in Transport.cpp, for example:

void Transport::UpdatePlayerPositions()
{
}

It is being used without having a structure. However, I'm guess you missed that? Whatever happened, right now it doesn't do crap.

YES..... IT Works Perfectly... The rest of the script was added.

Thx

 

[Tommy]

Glad it works, marking as solved. Hit the thanks button if you find my post useful. \o/