Re: [vassalengine-devel] efficient drawing; bounding boxes and selection borders

[Rodney K. <rod...@co...>]

> Do you think it would be possible to promote ColoredBorder to a real
> Decorator?


This doesn't sound like a good idea.  For example, any collections that
store a GamePiece would have to be updated whenever the selection changed,
and there's no way to know what all those collections are.  I'm not
surprised you saw some odd problems when you tried it.

If we put add/remove methods in the Highlighter interface, then the behavior
would be implementation-specific, which is a little strange I guess.  An
alternative is to change Map.getHighlighter() so that it returns a
SelectionHighlighter explicitly.  A SelectionHighlighter that wraps a single
Highlighter instance behaves the same as the bare Highlighter (doesn't it?)
so this shouldn't change any behavior.

rk