[Mac, 2.8.2] - validator for combo box

[Yaron Tadmor]

> 
> YT> For the time being, all I was thinking of doing is a simple "if"
where
> YT> needed in the wxTextValidator to know whether it's dealing with a
> YT> combo-box or text edit control (pretty much like the generic
validator
> YT> does).
> 
>  I don't like the explicit checks using wxDynamicCast for all the
usual
> reasons and even though we do have quite a few of them in the code
> already,
> I'd like to avoid adding another one.

I agree. It's not the best design. However, I thought of doing a small
local change which works, rather than a major design change.

> 
> YT> As for creating a wxTextEntryIFace, I'm not 100% sure I understand
> your
> YT> design concept. Would this class derive from anything? How does it
> YT> relate to the validators?
> 
>  wxTextEntryIface wouldn't derive from anything, but both wxTextCtrl
and
> wxComboBox would derive from it. The validator code would then use a
> wxTextEntryIface pointer instead of wxWindow one as this would be
enough
> for its needs.
>

Sounds like a good idea.


So should I wait until you get around to it?