Lilori - Jan 23, 2014 11:08
[u][b]Use Last GIT of Hercules[/b][/u]
Reproduce:[list=1]
[*]Use your royal guard
[*]Use your skill Exceed Break
[*]Map server crashed successfuly.
[/list]
Mhalicot - Jan 23, 2014 11:51
[color=#009acd][font='trebuchet ms', helvetica, sans-serif]Using Clean last Git revision, Failed to reproduce, it ddnt. crashed after using Exceed Break 2x..[/font][/color]
[color=#009acd][font='trebuchet ms', helvetica, sans-serif]Maybe there something in your src modification that affects your map server causing it to crashed.[/font][/color]
[color=#009acd][font='trebuchet ms', helvetica, sans-serif][img]http://i41.tinypic.com/2805bg0.jpg[/img][/font][/color]
Lilori - Jan 23, 2014 12:47
that's weird,
last 30 minutes, i did with clean GIT, and got map server crash. -,-
pan - Jan 23, 2014 16:10
Couldn't reproduce either
EDIT:
Just checked following Malufett's instructions and could reproduce
This post has been edited by
pan
on Jan 27, 2014 3:38
Mhalicot - Jan 23, 2014 23:24
[color=#009acd][font='trebuchet ms', helvetica, sans-serif]pleas run [url="http://herc.ws/wiki/MSVC_Crash_Debugging"]http://herc.ws/wiki/MSVC_Crash_Debugging[/url][/font][/color]
malufett - Jan 25, 2014 13:29
it will crashed if you don't have weapons equipped...
:meow:
pan - Jan 27, 2014 4:07
Open db/re/skill_require_db.txt and find:[code=auto:0]
2316,0,0,24:28:32:36:40,0,0,0,99,0,0,none,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0 //LG_EXEEDBREAK#Exceed Break#
[/code]Replace it with:[code=auto:0]
2316,0,0,24:28:32:36:40,0,0,0,2:3:4:5,0,0,none,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0 //LG_EXEEDBREAK#Exceed Break#
[/code]Also, open your src/map/skill.c and find:[code=auto:0]
case SC_EXEEDBREAK:
val1 *= 150; // 150 * skill_lv
if( sd && sd->inventory_data[sd->equip_index[EQI_HAND_R]] ) { // Chars.
val1 += (sd->inventory_data[sd->equip_index[EQI_HAND_R]]->weight/10 * sd->inventory_data[sd->equip_index[EQI_HAND_R]]->wlv * status->get_lv(bl) / 100);
val1 += 15 * (sd ? sd->status.job_level:50) + 100;
} else // Mobs
val1 += (400 * status->get_lv(bl) / 100) + (15 * (status->get_lv(bl) / 2)); // About 1138% at mob_lvl 99. Is an aproximation to a standard weapon. [pakpil]
break;
[/code]Replace it with:[code=auto:0]
case SC_EXEEDBREAK:
val1 *= 150; // 150 * skill_lv
if( sd )
{
if( sd->equip_index[EQI_HAND_R] >= 0 && sd->inventory_data[sd->equip_index[EQI_HAND_R]] )
{
val1 += (sd->inventory_data[sd->equip_index[EQI_HAND_R]]->weight/10 * sd->inventory_data[sd->equip_index[EQI_HAND_R]]->wlv * status->get_lv(bl) / 100);
val1 += 15 * (sd ? sd->status.job_level:50) + 100;
}
}
else // Mobs
val1 += (400 * status->get_lv(bl) / 100) + (15 * (status->get_lv(bl) / 2)); // About 1138% at mob_lvl 99. Is an aproximation to a standard weapon. [pakpil]
break;
[/code]
This post has been edited by
pan
on Jan 27, 2014 4:09
Mhalicot - Jan 27, 2014 4:15
[color=#009acd][font='trebuchet ms', helvetica, sans-serif]^ its nice if your going to make a pull request for that. :meow:[/font][/color]
pan - Jan 27, 2014 4:21
Well, I have one pull request that's been in that list for a week now, I'll see if I can make one today c:
Also I've documented skill_require_db, figuring out how to edit that thing was a mess:
[url="http://pastebin.com/nvZmBkYh"]http://pastebin.com/nvZmBkYh[/url]
EDIT:
Already made pull request, but somehow it merged with my last one...
[url="https://github.com/HerculesWS/Hercules/pull/254"]https://github.com/HerculesWS/Hercules/pull/254[/url]
This post has been edited by
pan
on Jan 27, 2014 4:31
Mhalicot - Jan 27, 2014 4:45
[color=#009acd][font='trebuchet ms', helvetica, sans-serif]^ Haha thats nice, lets wait for devs to take a look at it.[/font][/color]
malufett - Feb 10, 2014 10:14
Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414
Thanks pan..
:meow:
HisokaMachi - Mar 8, 2014 18:14
[quote name="malufett" timestamp="1392027282"]
Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414
Thanks pan..
:meow:[/quote]
can you link it again where i can see the fix
Angelmelody - Mar 8, 2014 18:42
[quote name="HisokaMachi" timestamp="1394302455"][quote name="malufett" timestamp="1392027282"]
Fixed @ 9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414
Thanks pan..
:meow:[/quote]
can you link it again where i can see the fix[/quote]
[url="https://github.com/HerculesWS/Hercules/commit/9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414"]https://github.com/HerculesWS/Hercules/commit/9b5bb9aa894d63d1d9f2cafa91f90bc4922bf414[/url]