Would be extremely cool if the UndoManager::perform and UndoableAction::perform/undo return the handy juce::Result class instead of a meaningless boolean. This way we can avoid to dynamic_cast each action to know what has gone bad in the last action.
makes sense ?