Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Config in Old Style/New Style of devotion. #209

Closed
wants to merge 2 commits into from
Closed

Added Config in Old Style/New Style of devotion. #209

wants to merge 2 commits into from

Conversation

Jedzkie
Copy link
Contributor

@Jedzkie Jedzkie commented Oct 27, 2013

Old Style: The reflect damage will return in redeemer.

New Style: The reflect damage will return in damager.

Credits to brathena.

Signed-off-by: Jedzkie jedzkie13@rocketmail.com

Old Style: The reflect damage will return in redeemer.

New Style: The reflect damage will return in damager.

Credits to brathena.

Signed-off-by: Jedzkie <jedzkie13@rocketmail.com>
@Jedzkie
Copy link
Contributor Author

Jedzkie commented Oct 27, 2013

Oh my, i got compile errors :(

status_change_end(target, SC_CHASEWALK, INVALID_TIMER);
status_change_end(target, SC_CAMOUFLAGE, INVALID_TIMER);
status_change_end(target, SC__INVISIBILITY, INVALID_TIMER);
status_change_end(target, SC_DEEP_SLEEP, INVALID_TIMER);
if ((sce=sc->data[SC_ENDURE]) && !sce->val4 && !sc->data[SC_LKCONCENTRATION]) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did u changed spaces for tabs or vice versa?

@Jedzkie
Copy link
Contributor Author

Jedzkie commented Oct 27, 2013

yes, i changed it.

@EPuncker
Copy link
Contributor

devs will kill u :P they like tabs since most of the code use tab for indentation and you changed it for spaces, but lets wait them to say, I was just stating this because they told me already to not do this when doing pull requests

@Jedzkie
Copy link
Contributor Author

Jedzkie commented Oct 27, 2013

oh i misunderstood your question, i use tab for that, i know devs use tabs LOL

@EPuncker
Copy link
Contributor

Nvm then xd
Em 27/10/2013 13:33, "Frost" notifications@github.com escreveu:

oh i misunderstood your question, i use tab for that, i know devs use tabs
LOL


Reply to this email directly or view it on GitHubhttps://github.com//pull/209#issuecomment-27171334
.

Fixed compile warnings
@Jedzkie
Copy link
Contributor Author

Jedzkie commented Oct 27, 2013

I fixed now the compile warnings, hope the devs will merge my pull req :)

@Jedzkie
Copy link
Contributor Author

Jedzkie commented Nov 8, 2013

can someone test this please? thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants