On Thu, Jun 06, 2013 at 09:16:52PM +0200, Peter Bex wrote:
> On Thu, Mar 14, 2013 at 05:43:46AM +0100, Florian Zumbiehl wrote:
> > Check the alist passed to alist-update is actually a pair before
> > using ##sys#slot on it.
> 
> This one almost slipped through the cracks.  It's a trivial change, so
> I decided that a second signoff is not neccessary, and pushed it
> immediately.

And of course only now I noticed the preceding discussion.... (-(

It didn't really lead to anything, and adding the check in front
would cause it to traverse the list twice.  I think I prefer the
traverse-once over check-and-traverse even if this would catch
some more error situations.

If you disagree, feel free to continue the discussion and revise.

Cheers,
Peter
-- 
http://www.more-magic.net

_______________________________________________
Chicken-hackers mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/chicken-hackers

Reply via email to