Add SAVE_LOG_IN_TEMP_INSTEAD_OF_PLUGIN_DIR preprocessor definition ...#9
Add SAVE_LOG_IN_TEMP_INSTEAD_OF_PLUGIN_DIR preprocessor definition ...#9pinodev wants to merge 1 commit intop0358:masterfrom
Conversation
…d code, disabled by default in order to remain backward compatible for now... # legacy-bckcompat-isofunc-addonly-preproc-toggle-hotfix # prepared-hotfix-toggle
|
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. |
|
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. |
... 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: