SublimeText/CTags — pull request #99 by hanoii (2012-12-04 18:15:29+00:00)
Title: show symbols multi

I believe the logic behind show symbols multi was wrong. I couldn't make it to work and after adventuring myself into the code I found that there was definitely issues with the code.

As far as I could tell, the function tagged_project_files was not serving its purpose as the project variable was set to None. This is probably because this was gone on the sublime text API.

This pull request changes the way show symbols (multi works). Instead of trying to get all the paths, it basically just lists everything what's on the tag file.

There was several ways I could have accomplished this, and although the logic might use a bit of cleanup, I tried to keep the pull request simple and related to the current codebase.

Basically for show symbols (multi) I am returning an empty list from files_to_search, and when get is invoked with an empty field, all lines are returned.

In the process, I changes multi for all in the menu, I believe it's more colloquial as well as in sync with the command pallet.

Also, I found reference to a lang argument in the symbols code but not used whatsoever. I believe it worth removing it if it's not used. This can be followed up on a separate thread and I don't mind cleaning it up if you want to.