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

Refactor String path -> Path path #111

Open
Mession opened this issue Sep 2, 2015 · 3 comments
Open

Refactor String path -> Path path #111

Mession opened this issue Sep 2, 2015 · 3 comments

Comments

@Mession
Copy link
Member

Mession commented Sep 2, 2015

Instead of strings for paths, we should prefer java.nio.file.Path.

@nimirum
Copy link

nimirum commented Sep 2, 2015

I can try to do this

nimirum added a commit to nimirum/tmc-netbeans that referenced this issue Sep 2, 2015
@jamo
Copy link
Member

jamo commented Sep 2, 2015

You probably should be aware of the work done here: testmycode/tmc-core#70 even though this work shouldn't collide.

@ljleppan
Copy link
Member

ljleppan commented Sep 4, 2015

NB: Prefer usage of java.nio instead of java.io.

That is, use Files.exists(Path path)* instead of path.toFile().exists() etc. and preferably replace any existing usage of the class File with Path.

*) http://docs.oracle.com/javase/7/docs/api/java/nio/file/Files.html.

jamo added a commit that referenced this issue Oct 12, 2015
Pathy Strings refactored to Paths, closes #111
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants