Skip to content

Allow .xml in map names#236

Merged
kazimuth merged 3 commits into
masterfrom
mapnames
Jan 21, 2016
Merged

Allow .xml in map names#236
kazimuth merged 3 commits into
masterfrom
mapnames

Conversation

@axc
Copy link
Copy Markdown

@axc axc commented Jan 21, 2016

Resolves #235

Comment thread src/main/battlecode/server/Server.java Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This logic should be in GameMapIO.

@axc
Copy link
Copy Markdown
Author

axc commented Jan 21, 2016

Updated.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

mapName.lastIndexOf('.')?

@axc
Copy link
Copy Markdown
Author

axc commented Jan 21, 2016

Updated again according to comments, and added unit test

kazimuth added a commit that referenced this pull request Jan 21, 2016
@kazimuth kazimuth merged commit f249c24 into master Jan 21, 2016
@kazimuth kazimuth deleted the mapnames branch January 21, 2016 22:03
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.

3 participants