[PD-dev] popup,...

[IOhannes m zmoelnig]

hi ben, hi list

i just checked your [popup], like it and have submitted several (imo) 
improvements to the CVS.
namely:
- no restriction for max# of "options"; this might be not very 
interesting (as on my machine i can only display 40 options at one time, 
so 100 seems to be sufficient anyhow) BUT i added additionally checks 
whether the max# has been exceeded; this keeps [popup] from segfaulting 
when you have more than 100 options

- using a mixture of user-defined and default values if you provide less 
than 5 arguments (i thought that 5 args are a bit much to remember)

- removed the "loadbang"-constraint (you can now add options and select 
entries before the object has been drawn the 1st time)

since i only tested it on linux, it would be great if somebody could 
tell me what i've broken...


but i have some questions:
- why are there 2 inlets ? i noticed you uncommented the 
class_addsymbol()-method, so it seems like there is some reason for not 
using the 1st inlet for both numeric and symbolic selection of elements.

- even more puzzling: why are there 2 outlets ?
for clarification, i don't have any problems with separate outlets for 
typfixed outlets (1 for numeric indices and 1 for symbolic names), but 
you are using the first outlet for 2 types (float vs. bang) so you 
somehow need a route to segregate those types.
i would've suggested using either 1 outlet (with either ("bang" and 
"float" and "symbol") or ("bang" and "list")

since now the "loadbang" seems to work, could the "bang" output be 
removed ? (i find it rather annoying that everytime i switch console 
(this is: unhide the window), a bang is emitted.


but i guess, this would break a lot of patches....


mfg.as.dr
IOhannes