segfault in rev 640

**gaeorn** · #1 06-08-2009, 03:50 PM

No more segfault after reverting that change.

**KLS** · #2 06-08-2009, 06:16 PM

I'll commit the revert then, thanks.

Edit: Actually I changed it up a bit rather than a full revert because I realized a full revert would take out some required functionality... With any luck the crash will be gone and everything will work.

**gaeorn** · #3 06-09-2009, 04:09 AM

No crash with that change.

Also, I specifically tried out the adventurer's stone and it is scaled based on number of wins.

One odd thing I noticed is I set a GM char on my server to enough wins to max the stats. With that done, the adventurer's stone was showing 108 hp and mana. I'm guessing there's a rounding problem with the math.

**KLS** · #4 06-09-2009, 04:17 PM

Probably a problem with the script, I know I had it set to 1.0 in the script arbitrarially earlier and it worked fine.

**gaeorn** · #5 06-09-2009, 05:21 PM

You are correct:

Code:

Index: quests/items/LLDAugText.pl
===================================================================
--- quests/items/LLDAugText.pl  (revision 88)
+++ quests/items/LLDAugText.pl  (working copy)
@@ -22,7 +22,7 @@
         
         if($ruj_wins > 74)
         {
-                $guk_wins = 74;
+                $ruj_wins = 74;
         }
 
         if($tak_wins > 78)

**cavedude** · #6 06-09-2009, 07:48 PM

It's OK, KLS I misspelled "warrior" in Gram's quest file to obtain the PoP charm.

The change to the adventure's stone will come down with the next quest svn update, thanks gaeorn.

**gaeorn** · #7 06-09-2009, 08:50 PM

Heh, not like I did much. KLS did all the thinking... I just looked where she pointed.