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

Remove Game.ModData references from Widget code. #10788

Merged
merged 3 commits into from Feb 25, 2016

Conversation

pchote
Copy link
Member

@pchote pchote commented Feb 20, 2016

The main focus of this change is code hygiene (relying on a static reference for something which may change isn't nice), but has a side-effect of adding support for map-defined resources in the editor.

this.onStart = onStart;
this.onExit = onExit;

Copy link
Member

Choose a reason for hiding this comment

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

Lonely tab.

@pchote
Copy link
Member Author

pchote commented Feb 20, 2016

Fixed.

@Mailaender
Copy link
Member

👍

@obrakmann
Copy link
Contributor

OpenRA.Mods.Common/Widgets/Logic/SettingsLogic.cs:L28: [SA1215] All non-static readonly private fields must be placed before all non-static non-readonly private fields.

OpenRA.Mods.Common/Widgets/Logic/SettingsLogic.cs:L29: [SA1215] All non-static readonly private fields must be placed before all non-static non-readonly private fields.

@pchote
Copy link
Member Author

pchote commented Feb 20, 2016

Fixed.

@obrakmann
Copy link
Contributor

Needs a rebase as well

@pchote
Copy link
Member Author

pchote commented Feb 23, 2016

Rebased.

@obrakmann
Copy link
Contributor

Looks good to me, 👍

obrakmann added a commit that referenced this pull request Feb 25, 2016
Remove Game.ModData references from Widget code.
@obrakmann obrakmann merged commit fe6c4d8 into OpenRA:bleed Feb 25, 2016
@obrakmann
Copy link
Contributor

Changelog

@pchote pchote deleted the widget-moddata branch February 27, 2016 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants