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

Infinite loop with berserk weapon special if value is negative #3918

Closed
wants to merge 7 commits into from

Conversation

newfrenchy83
Copy link
Contributor

In weapon special berserk value by default is 1 and value less to 1 are exclude, i must be do what is the same in berserk ability.

If in attack end in FORCE_CHANCE_TO_HIT macro, unit or second unit doesn't exist anymore because rare case where unit disapear before attack end event was calledthen we have an error message like "Invalid WML found: [unstore_unit]: variable 'second_unit' doesn't exist" because presence of unit not condition of execution of event.
In weapon specila berserk value by default is 1 and value less to 1 are exclude, i must be do what is the same in berserk ability.
@newfrenchy83 newfrenchy83 changed the title Fix berserk ability could used value min to 1 berserk ability value min 1 same if cfg value<1 Feb 21, 2019
@jostephd jostephd added the Needs more info Issues deemed to contain insufficient information for reproducing or fixing. label May 19, 2019
@jostephd jostephd changed the title berserk ability value min 1 same if cfg value<1 Infinite loop with berserk weapon special if value is negative May 20, 2019
@jostephd jostephd removed the Needs more info Issues deemed to contain insufficient information for reproducing or fixing. label May 20, 2019
@jostephd
Copy link
Member

Reopening for this part:

Question to developers: What should [berserk] value=-1 [/berserk] do?

@jostephd jostephd reopened this May 20, 2019
@newfrenchy83
Copy link
Contributor Author

i pull tis PR #4086 for resolve special and abitities, because of 3 other files changed after an error to my part.

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

3 participants