[XForms] Possible bug in xforms 1.2.4

[ternaryd]

In file popup.c, function

   fli_popup_insert_items()

there is a static declaration of variable
"val". While one can argue if this is good
programming practice, I do believe that this
variable should be automatic and initialized
by popup->counter, which is reset just before
this call. Isn't that the purpose of the reset
counter function?

Using the public function fl_set_nmenu_items(),
the value is always increased making it
difficult to identify the items by their
position.