Jump to content

Dark Theme was set as default for all members. Please report any issues here.

Sign in to follow this  
bWolfie

Star Gladiator LOV/Parrying SL

Recommended Posts

Description: When a Star Gladiator is Soul Linked, the 'Lord of Vermilion' and 'Parrying' skills become available for them to use.

This must be the most requested source edit of all time? Idk why. Anyway, here you go - in plugin form.

https://github.com/bWolfie/HPM-Plugins/blob/master/src/plugins/sg_parrying_lov.c

Edited by bWolfie

Share this post


Link to post
Share on other sites

Thanks for sharing this Myriad! I hope you can help us about the @pk off and on plugin too. It's a most requested to modify too. Dastgir's pk plugin is a good one too but it cannot be used on a PK Server. If you want an explanation, I will.

Share this post


Link to post
Share on other sites

Update:

- skill_get_requirement_post was missing a 'sd' check. 
- Configuration for which skills become available is now set in an array.
- Console will print a warning if the array is not configured correctly.

// Configuration
int extra_skill[] = {
	// Add the skills/levels here
	// Format: skill_id, skill_lv,
	WZ_VERMILION, 10,
	LK_PARRYING, 10
};
// End Configuration

 

Edited by Myriad

Share this post


Link to post
Share on other sites
In file included from sg_parrylov.c:23:0:
sg_parrylov.c: In function 'plugin_init':
../plugins/HPMHooking.h:54:11: error: 'HPMHOOK_post_pc_calc_skilltree_clear' undeclared (first use in this function)
   (void)((HPMHOOK_post_ ## ifname ## _ ## funcname)0 == (hook)), \
           ^
sg_parrylov.c:75:3: note: in expansion of macro 'addHookPost'
   addHookPost(pc, calc_skilltree_clear, pc_calc_skilltree_clear_post);
   ^
../plugins/HPMHooking.h:54:11: note: each undeclared identifier is reported only once for each function it appears in
   (void)((HPMHOOK_post_ ## ifname ## _ ## funcname)0 == (hook)), \
           ^
sg_parrylov.c:75:3: note: in expansion of macro 'addHookPost'
   addHookPost(pc, calc_skilltree_clear, pc_calc_skilltree_clear_post);
   ^
../plugins/HPMHooking.h:54:52: error: expected ')' before numeric constant
   (void)((HPMHOOK_post_ ## ifname ## _ ## funcname)0 == (hook)), \
                                                    ^
sg_parrylov.c:75:3: note: in expansion of macro 'addHookPost'
   addHookPost(pc, calc_skilltree_clear, pc_calc_skilltree_clear_post);
   ^
../plugins/HPMHooking.h:54:64: warning: left-hand operand of comma expression has no effect [-Wunused-value]
   (void)((HPMHOOK_post_ ## ifname ## _ ## funcname)0 == (hook)), \
                                                                ^
sg_parrylov.c:75:3: note: in expansion of macro 'addHookPost'
   addHookPost(pc, calc_skilltree_clear, pc_calc_skilltree_clear_post);
   ^

@Myriad I got this when i compile..

Share this post


Link to post
Share on other sites
3 hours ago, Myriad said:

Have you edited file at all? Using updated Hercules? I am using updated Hercules and it works fine.
bTjhSjo.png

Well our src revision is not the latest.. we are using this.. https://github.com/shadow-mo0n/Hercules

Edit:
Its working now. I just applied the updates for the HPM Hooking and the splitted pc_calc_skilltree.

Edited by Rebel

Share this post


Link to post
Share on other sites
On 6/30/2018 at 10:13 AM, Bringer said:

do you have src code of this can be use on rathena?

under version I already link the code it is sourced from
also rathena noob. moreso than habnoobis. use herc. thank.

Edited by Myriad

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
Sign in to follow this  

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.