search

ameyp / CscopeSublime

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

Only one ST plugin can have mousemap files because ST doesn't support co... #17

Closed vanrijn closed 11 years ago

vanrijn commented 11 years ago

...ntexts in mousemap files. Removing mousemap files until we can find a real solution.

I'm not sure if I'm doing this right, but I wanted you to review my changes before merging them into master. =:)

ameyp commented 11 years ago

But why not leave it in as a best-effort piece of code? If the user doesn't have any other mousemap files, it will work just fine. If the user has other mousemap files, it won't. This seems better to me than just removing the functionality completely. Perhaps we could add a note in the README that mentions this in order to avoid any confusion?

vanrijn commented 11 years ago

The problem is that I want this functionality from more than one plugin, and if any one plugin ships with a mousemap that eats double-click, it completely breaks being able to do that. I am going to try to write a simple plugin to allow this functionality for all plugins (see http://www.sublimetext.com/forum/viewtopic.php?f=6&t=11680), but before that can happen, I have to remove the mousemap files from Cscope, as FichteFoll noted.

vanrijn commented 11 years ago

And it's not that it won't work if another plugin ships with a mousemap file. It's that you have no idea which plugin will win the war over double-click handler.

ameyp commented 11 years ago

Fair enough, that makes sense.