Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[platform-ui-dev] IUndoableOperation spec cleanup/tightening

While everyone is having fun at EclipseCon, I'm using the quiet time to do some spec cleanup and tightening.
This is just a heads up that I've added some assertions in AbstractOperation to prevent setting of null operation labels, and updated the spec of IUndoableOperation to make it clear that null labels aren't allowed.

Up to now, using a null label would result in an NPE way downstream, typically when the user tried to look at the undo menu with such an operation in the history. Now you'll see the failure early. Implementors of IUndoableOperation that don't use the AbstractOperation implementation will continue to behave the same.

See https://bugs.eclipse.org/bugs/show_bug.cgi?id=218467 for details.

susan


Back to the top