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

Add SAVE_LOG_IN_TEMP_INSTEAD_OF_PLUGIN_DIR preprocessor definition ... #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pinodev
Copy link

@pinodev pinodev commented Apr 29, 2019

... and code, disabled by default in order to remain backward compatible for now... # legacy-bckcompat-isofunc-addonly-preproc-toggle-hotfix # prepared-hotfix-toggle

For the motivations behind this changeset, see:

…d code, disabled by default in order to remain backward compatible for now... # legacy-bckcompat-isofunc-addonly-preproc-toggle-hotfix # prepared-hotfix-toggle
@p0358
Copy link
Owner

p0358 commented May 6, 2019

Can you elaborate the benefits of this change? I think that as long as the logger tries to write to the same file, the issue with errors may persist.
As for excessive disk usage (which will too be fixed in next version in case of unrecognized extension, and will only happen in debug mode), the log file is deleted on each Notepad++ start in order to save space, so this wouldn't change anything.
Solution I am currently implementing will simply show one message box with error in a session, and then will ignore the other errors to let user continue their work with no interruptions.

@pinodev
Copy link
Author

pinodev commented May 10, 2019

Well, the work is not really over: this should probably a configuration option instead of a compiler switch.

The advantage of putting the log file in TEMP, is that it reduces the strain on the AppData directory. In my case, there were really many many MANY writes in the log file. And each of this operations is opening the file, writing to it and closing it.

I am fairly nomadic, and I use Notepad++ in a portable manner; in particular, my AppData/Notepad++ directory is synchronized.

Having the CodeStats plugin writing at this high rate in a file that is being monitored and synchronised caused a problem I've never had before.

I admit this is an edge case, but if you think of it, the log file is not relevant for the functioning of the application anyway, it's a little bit of information and a lot of noise. For the greater good, I had to get rid of it, and move it to the TEMP folder.

It was a easy and effective fix.

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