tweezy23 / anyedittools

Automatically exported from code.google.com/p/anyedittools
2 stars 0 forks source link

Self assigned key bindings for compare/replace doesn't work in Eclipse 3.6 #48

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Set Ctrl+Alt+O as key binding for "Compare with Opened editor" command. When 
: in Windows
2. While editing a source, press Ctrl+Alt+O

What is the expected output? What do you see instead?
Nothing happened !

What version of the product are you using? On what operating system?
Eclipse 3.6
Anyedit 2.3.2.201010252259

Please provide any additional information below.

Original issue reported on code.google.com by cyril.chaize on 27 Oct 2010 at 2:49

GoogleCodeExporter commented 9 years ago
Thanks for the report!
It is broken on Eclipse 3.6 only. 3.5 just works. Right now I have no clue what 
the guys at Eclipse changed in 3.6 that it just stopped to work... I see in the 
debugger that none of the action's lifecycle methods are called on the key hit, 
it means my code is simply not executed at all... Cool. Diving into the debug 
session...

CU,
Andrei

Original comment by iloveeclipse on 27 Oct 2010 at 8:55

GoogleCodeExporter commented 9 years ago
Seems that fix for this one is the root cause:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=300526

I have no default handler on compare/replace actions as I never needed one. 
They've silently broken API contract. Cool.

Original comment by iloveeclipse on 27 Oct 2010 at 9:14

GoogleCodeExporter commented 9 years ago
Fix committed to head. 

I will release an updated version this weekend (may be). Feel free to build it 
from source if you need the patched version ASAP.
Thanks for reporting this!!!

Regards,
Andrei

Original comment by iloveeclipse on 27 Oct 2010 at 9:50

GoogleCodeExporter commented 9 years ago
You're welcome Andrei.
Your plugin is very useful and practical. It's magic !

Original comment by cyril.chaize on 28 Oct 2010 at 7:44

GoogleCodeExporter commented 9 years ago
I have updated this morning and it's working now.
Thanks

Original comment by cyril.chaize on 2 Nov 2010 at 10:37