Description Douglas Pollock 2005-02-25 09:37:41 EST

I have realized that acceleratorSets didn't work in 3.0.  They are not parsed
correctly by the Persistence class.

The acceleratorSets extension point was introduced in 2.0, and deprecated in
2.1.   It has been replaced with the commands extension point.  The extension
point was parsed correctly in 2.1.x and earlier.

I believe there are three separate options.
1.) Go back to the 2.1.x code and find the acceleratorSets parsing code.  Copy
it into 3.1.
2.) Remove the acceleratorSets extension point and mark it as a breaking change
in 3.1.
3.) Leave the extension point, but leave do not provide any code to parse it. 
Wait for someone to complain.


My general impression is to go for (2).  There are no bugs filed about its
absence yet, and my feeling is that no one misses it.  By removing the extension
point, anyone who happens to use it would see compiler errors.  The note about
breaking changes would give people using it a pointer as to where they should
look.  (As a note, it does not break binary compatibility -- referencing
extension points that don't exist does not cause errors at run-time.)


For now, I'm going implement (3), and wait for some more input on this bug.