benmatselby / sublime-phpcs

🔍 PHP CodeSniffer, PHP Coding Standard Fixer, Linter, and Mess Detector Support for Sublime Text
Other
812 stars 126 forks source link

Crash/Freeze when saving user settings file in 3049+ #86

Closed Narretz closed 11 years ago

Narretz commented 11 years ago

When I do the above, Sublime freezes indefinitely. Only way out is to kill it. I don't have to change the values, just saving it is enough. It also happens when I add / remove the phpcs from the ignored packages list. The file is actually saved, and the last thing the console logs is that the file is being reloaded. No difference between atomic_save true / false

Win 7 64bit Sublime 3049+

I also reported this in the Sublime Forum.

benmatselby commented 11 years ago

Thanks for flagging this. I believe 3050 is a night build? I cannot replicate this in the latest beta version: 3047, so will need to wait until the next full beta release of Sublime Text.

Narretz commented 11 years ago

Yeah, I am on the dev channel. Maybe it's related to the many changes to file change detection. Btw, does Sublime have a real crash log?

Narretz commented 11 years ago

Sorry, I was to quick with this. I actually cannot save any package settings file. It's not phpcs specific.

Narretz commented 11 years ago

I keep this closed, but as it turns out, this is 100% reproducible with 3049+. It's probably a Sublime bug with reloading the settings file, and it may affect other plugins too, but I wasn't able to reproduce it reliably. I updated the description in the start post.

Narretz commented 11 years ago

Fixed in 3052

benmatselby commented 11 years ago

Excellent, thanks for letting me know @Narretz