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

3319: Fixing Unofficial On Map Predesignate Invalid Turn Processing NPEs #3414

Merged
merged 8 commits into from
Jan 31, 2022

Conversation

Windchild292
Copy link
Contributor

This fixes #3319.

The first three commits are cleanups, the fourth is the bugfix.

Copy link
Member

@NickAragua NickAragua left a comment

Choose a reason for hiding this comment

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

This needs a revision to the way it's approached.

@Windchild292
Copy link
Contributor Author

I did a deeper dive, and this was introduced by the changes in #2558. I've got a proper cause fix now, which also improves the performance of said unofficial option.

… hexes when using the unofficial on map predesignate option
@Windchild292 Windchild292 changed the title 3319: Fixing Princess::unloadTransportedInfantry NPE 3319: Fixing Unofficial On Map Predesignate Invalid Turn Processing NPEs Jan 29, 2022
@Windchild292 Windchild292 merged commit dcf9b0b into MegaMek:master Jan 31, 2022
@Windchild292 Windchild292 deleted the dev_Windchild_3319 branch January 31, 2022 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants