EQEmulator Forums

EQEmulator Forums (https://www.eqemulator.org/forums/index.php)
-   Archive::Development (https://www.eqemulator.org/forums/forumdisplay.php?f=621)
-   -   Starting Items Completed on 0.5.0DR3 (https://www.eqemulator.org/forums/showthread.php?t=9509)

krich 09-23-2003 05:34 PM

Starting Items Completed on 0.5.0DR3
 
I've completed the Starting Items for 0.5.0DR3. This is a long DIFF and my first time generating a diff, so here goes.

Command line on linux is: diff -uBb original-file updated-file > file.diff

There are three files changed: common/database.h, common/database.cpp, world/client.cpp

The diffs are located here:

client.cpp.diff

database.h.diff

database.cpp.diff


regards,

krich

Merth 09-24-2003 12:28 AM

Thanks for the work! I'm putting this as a sticky until we can get to it.

kanechart 09-24-2003 03:21 AM

Thanks bro good work :)

krich 09-24-2003 05:30 PM

Warning: This seems to totally screw up when compiled under windows. Works fine under Linux. Trying to figure that out at the moment.

Regards,

krich

DeletedUser 09-24-2003 05:54 PM

Hes here to stop win32 compiling, STOP HIM! :o

Thanks for the assistance :)

krich 09-30-2003 03:20 PM

Quote:

Hes here to stop win32 compiling, STOP HIM!
Bah, foiled again. It turns out not to be a Windows problem afterall. Looks like the crashing problem was some invalid data in the starting_items table.

Database::GetItem(int32) does not take kindly to being called with zero. Causes a segmentation fault for some reason. I couldn't figure out a good way of error checking before that call (I suspect zero is not the only value that will make this function segfault).

Anyway, don't put invalid itemids in your starting_items table and you'll be fine with this enhancement.

<bad-dubbing>Hehe...Image, my next enhancement will have unstoppable Linux kung-fu. Your victory will be short lived!</bad-dubbing>

Regards,

krich

Trumpcard 10-04-2003 02:28 AM

Ok, I merged this in this morning... Looks like it's working good with my database (using TCS's, but I think his starting items table is based off Drawdes and Lurkers) ... I did notice the 0 add segfault, so I wrapped to add calls if an if(myitem) to get around that..

Also, I modified some of your logging.

I've pushed this out to CVS, so should be ready for your verification.
Let me know if you see any problems with it, or changes that need to be made. I hand to hand merge the changes as the diffs were too far apart from the code base.

Thanks for the donation!

krich 10-05-2003 03:31 PM

Looks good.

Thanks,

krich

Trumpcard 10-05-2003 10:10 PM

Thanks krich.. God, I've never made so many typos at one time as I did in that last message!


All times are GMT -4. The time now is 11:51 PM.

Powered by vBulletin®, Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.