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]