Skip to content

Use path object so file separator is not needed, default to install p…#26

Open
jwlodek wants to merge 2 commits intoareaDetector:masterfrom
jwlodek:master
Open

Use path object so file separator is not needed, default to install p…#26
jwlodek wants to merge 2 commits intoareaDetector:masterfrom
jwlodek:master

Conversation

@jwlodek
Copy link
Copy Markdown
Member

@jwlodek jwlodek commented Nov 18, 2024

…roperty file in .config

Currently, the imagej NTNDA viewer will create the properties file in the home directory. It would be preferable if this was created in the .config folder instead.

import java.io.FileOutputStream;
import java.io.PrintStream;
import java.nio.file.Files;
import java.nio.file.LinkOption;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this line needed? I don't see LinkOption used in the code?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think it can be removed. I don't remember why I added it...

@MarkRivers
Copy link
Copy Markdown
Member

Can you remove it and test that it works?

@jwlodek
Copy link
Copy Markdown
Member Author

jwlodek commented Feb 13, 2025

image

I've tested it on my machine and it appears to work.

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.

2 participants