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

Make use of FileHelpers.NormalizeFilePathSeparators #2572

Merged
merged 1 commit into from May 18, 2014

Conversation

danzel
Copy link
Contributor

@danzel danzel commented May 18, 2014

First part of the FileHelpers things for #2555

This is an easy one that makes a nice cleanup, so submitting it by itself.

The remaining FileHelpers are mostly just used by StorageDevice, so maybe we leave them for now as #2389 is going to tackle them.
If you do plan on using the code in there, DirectoryGetFiles and DirectoryGetDirectories both ignore the given path on WINDOWS_STOREAPP.
It looks like this bug was copy/pasted out of StorageDevice.
(I'll send a PR to fix them when I get a chance.)

@danzel
Copy link
Contributor Author

danzel commented May 18, 2014

This depends on #2571 to be merged first, it adds using Microsoft.Xna.Framework.Utilities; to ContentReader.cs which this requires. Will bump after that has been merged.

@tomspilman
Copy link
Member

@danzel - You'll need to merge in the latest changes from develop into this branch first.

@danzel
Copy link
Contributor Author

danzel commented May 18, 2014

Yep, doing now :)

@tomspilman
Copy link
Member

Looks good... merging!

tomspilman added a commit that referenced this pull request May 18, 2014
Make use of FileHelpers.NormalizeFilePathSeparators
@tomspilman tomspilman merged commit 130ab44 into MonoGame:develop May 18, 2014
@danzel danzel deleted the normaizefilepathseparators branch May 19, 2014 08:56
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