Mailinglist Archive: yast-devel (47 mails)

< Previous Next >
Re: [yast-devel] 'remove' builtin: strange behavior?
  • From: Stanislav Visnovsky <visnov@xxxxxxx>
  • Date: Tue, 7 Oct 2008 10:26:14 +0200
  • Message-id: <200810071026.14430.visnov@xxxxxxx>
Dňa Friday 03 October 2008 12:23:49 Stanislav Visnovsky ste napísal:
Hi!

During bug fixing, Jozo came with interesting observation: the default
behavior of 'remove' builtin for maps if a map does not contain key, is
strange.

Documentation:
"Remove the value with the key KEY from a map. Returns nil if the key is
invalid."

The definition for list is similar. So, this means the following code will
break quite a bit, if "this key" does not exist.

a = remove( a, "this key" )

The proposal is to change the behavior to return the original (unchanged)
map/list for CODE11 (openSUSE >= 11.1). Seems code in SVN seems to be quite
OK (either uses haskey() or is safe by other means, or broken just because
of this behavior ;-))

Any comments or objections?

OK, so I'll change the behavior in trunk for both maps and lists.

Stano
--
To unsubscribe, e-mail: yast-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: yast-devel+help@xxxxxxxxxxxx

< Previous Next >
References