Compile Errors in Rev668
I attempted to compile Rev668 twice and got these errors both times.
Error 8 error C2664: 'void Client::Message_StringID(int32,int32,int32)' : cannot convert parameter 2 from 'const char [18]' to 'int32' c:\Program Files (x86)\EQEmu Source\EQEmu Source\zone\client_packet.cpp 2020 Error 9 error C2662: 'Client::GetClientVersion' : cannot convert 'this' pointer from 'const Client' to 'Client &' c:\Program Files (x86)\EQEmu Source\EQEmu Source\zone\client_mods.cpp 44 |
Should be fixed now.
|
Sorry about the compile errors I have caused lately lol. I swear I test them on Linux before committing, but apparently Windows seems to be more sensitive about some things than Linux is. I have seen compile errors from a few people lately, but I think it is just due to the big changes that we have all be making lately. There are bound to be mistakes and between testing and compiling the code and editing it before committing, sometimes something small can get missed. I always try to make sure that the very last thing I do before committing is to run one last compile with every change in to be as sure as possible that it is all good.
I guess the good thing is that most of the compile issues aren't too hard to fix, and get resolved really quickly. I have probably fixed 2 revisions from others that had compile issues recently. Not a big deal IMO, though there is always room for improvement :) Just making sure no one is getting too upset over the compile issues lately, lol. |
Minor inconveniences, don't let it bother you. Though, it was a nice change. Usually compile errors only effect Linux boxes around here!
|
Quote:
|
Rev 670 compiles fine, however crashes on ZONE.EXE
Code:
|
The code changes KLS made fixed the compile problem and it is working fine now. Trevius don't let a few compile errors bother you those can happen to anyone at anytime.
Zandig, I got a crash error with zone.exe as well with revision 669 and it was just a compatibility issue between the database and the emulator. Updating to the newest peq database fixed the problem. |
I usually learn something new whenever it happens, so It's all good :cool:
|
Quote:
|
All times are GMT -4. The time now is 10:15 PM. |
Powered by vBulletin®, Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.