ameyp / CscopeSublime

A Cscope plugin for Sublime Text 2 and 3
89 stars 37 forks source link

Possibility to add all results to a list and navigate through them? #22

Closed ysheng26 closed 8 years ago

ysheng26 commented 11 years ago

Hi,

first of all thank you very much for this nice plugin.

Is it possible to add all the results into the history list and navigate through the list via ctrl+shift+[ ?

the current implementation won't clear the list when querying new symbols, and I can't afford manually hitting enter on every result.

btw, is it possible to update cscope_out automatically?

ameyp commented 11 years ago

About your first suggestion:

I thought of doing that, but then I figured that the current behaviour is way more flexible. In its current implementation, you can have one search results window open for a global definition lookup, another open for a reference lookup, and you can navigate both independent of each other. Adding search results to a list that you can navigate through with the keyboard would only make sense if we allow the user to have only one type of search results open at any given time.

The second suggestion is doable and should be rather simple. However I'm no longer actively developing the plugin, so if @vanrijn doesn't mind making the changes, this could be added. Or if you know python, you could do it, we welcome all pull requests :)

vanrijn commented 11 years ago

I'm not sure I understand what you mean about updating cscope_out automatically. Can you explain? And yeah, I agree with @ameyp about not limiting the user to only having one type of search results open at any given time. I have about 4 different cscope search tabs open right now. =:)

ameyp commented 11 years ago

@vanrijn I think he means adding a command to create/recreate cscope.out instead of having the user create the cscope database file himself.

ysheng26 commented 11 years ago

@ameyp @vanrijn Yes, actually I mean if a update cscope.out command be able to run in the background since the file has changed?

vanrijn commented 8 years ago

Okay, so the first idea I'm not going to try to implement. I'm open to reviewing patches and pull requests if someone else wants to implement this in a clean way. But I prefer the way this plugin currently works (a single results buffer). As for the second idea, it was a great idea and I included this functionality in a recent commit. So this can be closed now. Thanks for the ideas! =:)