[CinCV TNG] [PATCH] New base class Selection

Johannes Sixt j6t at kdbg.org
Mon Aug 28 23:11:46 CEST 2017


Am 28.08.2017 um 21:43 schrieb Einar R√ľnkaru:
> Hi.
> 
> Selection is a textbox with popup menu of values
> The derived classes hold tables of internal values
> with corresponding strings shown to user.
> To add or remove a value only the table needs change.

It is desirable to have such a thing.

But, oh boy, the name! You are squatting such a generic name, 
"Selection", for something that is not a "selection". Please make it 
less generic, such as SelectionTextBox.

(Yeah, I know, there are two really hard things in software engineering: 
cache invalidation, naming things, and off-by-one errors...) ;)

-- Hannes


More information about the Cinelerra mailing list