mikaeljorhult / brackets-todo

Brackets/Edge Code extension that displays all TODO comments in current document or project.
144 stars 37 forks source link

Crashing in large project when scope not explicitly set to "current" #103

Closed kenherbert closed 9 years ago

kenherbert commented 9 years ago

I'm on Windows 7 64-bit using Brackets 0.44 and Brackets Todo 0.8.0, and I have a very large project (over 1200 files in total).

When switching to this folder, or when opening Brackets with this folder already set as the working location, Brackets locks up for a few seconds, then one of the two processes crashes (having reached over 500Mb memory usage), leaving the shell window visible with no functionality.

This occurs both when I have a .todo file with scope set to "project" (understandable given the number of files, just stating it for clarity) but it also happens with no .todo file present. Having read similar issues it seems that scope: "current" should be the default when no .todo file is found, but this doesn't seem to be the case.

Explicitly creating a .todo file and setting the scope to "current" fixes the issue, but users really shouldn't have to create a .todo file in every folder of their computer.

anzhihun commented 9 years ago

Brackets Todo 0.8,0 allows you configure scope through configure dialog which can be opened by clicking the configure button on right of the Todo's toolbar. Then it will take effect to all the projects. .todo file only takes effect to the project it belongs to.

kenherbert commented 9 years ago

Thanks for that. I must have enabled project scope in the configuration a while ago and forgotten about it.