Define constants for Tool onCancel reasons

[Eneroth3]

Currently we need to check for hardcoded integer values, or define these somewhere in our extensions. Ideally these should be defined by the SketchUp Ruby API.

CANCEL_ESC = 0
CANCEL_RESELECT = 1
CANCEL_UNDO = 2

[thomthom]

Yes! I'm always defining these in my extensions!

[thomthom]

We should do a "Constant all the Magic" cleanup sweep.

[DanRathbun]

Since these are relative to tool objects, ... I'd prefer ...

TOOL_CANCEL_ESC = 0
TOOL_CANCEL_RESELECT = 1
TOOL_CANCEL_UNDO = 2

... or ...

TOOL_REASON_ESC = 0
TOOL_REASON_RESELECT = 1
TOOL_REASON_UNDO = 2

[Fredosixx]

A few remarks

• The value 1 is never received?
• The value 2 is received for Undo and Redo

[DanRathbun]

Just a thought that if Sketchup::Tool were to be defined (instead of being abstract) as:

class Sketchup::Tool

  CANCEL_REASON_ESC = 0
  CANCEL_REASON_RESELECT = 1
  CANCEL_REASON_UNDO = 2
  CANCEL_REASON_REDO = 3

end

... then any explicit subclassing would inherit the constants.

This still gives programmers the option of subclassing or not.

Backstory: I dislike so many top level constants. The list of them is already too long.