[rkward-cvs] [rkward] rkward/plugins/00saveload/save: oops, setting an empty label for <browser> also made .addFromUI() produce an empty variable name and by that an infinte loop. maybe add a check to the JS function?

[Thomas Friedrichsmeier]

Hi,

On Thursday 04 December 2014 14:41:01 m. eik michalke wrote:
> oops, setting an empty label for <browser> also made .addFromUI() produce an
> empty variable name and by that an infinte loop.

hm. Two issues, here:
1) When parsing a command like 
  list(""=1)
R seems to resort to a strange hack to make this fail. In essence, this causes 
R to jump back to its top-level loop, while we did not expect it (other parse 
errors are handled in a less drastic manner). Should now be fixed, i.e. only a 
regular error, not RKWard going into an endless loop. Fortunately this only 
affected commands that are _not_ treated as user-commands in RKWard.

2) I got confused about where the label comes from. What I meant to suggest 
was rather: Keep the label on the <browser>, drop the "Save to"-label from the 
<frame> around it. That would allow you to continue using addFromUI(). 
However, it really does not matter, either way.

Regards
Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/rkward-tracker/attachments/20141204/a71d8cdf/attachment.sig>