> I just noticed the function I wrote was replaced with
>
> (dolist (key keys plist)
> (remf plist key))
>
> but that does not match the behavior of remove-from-plist,
>
> CL-USER> (alexandria:remove-from-plist (list :a 1 :b 2 :a 3) :a)
> (:B 2)
> CL-USER> (alexandria:delete-from-plist (list :a 1 :b 2 :a 3) :a)
> (:B 2 :A 3)
>
> It also don't check for malformed plists. The commit message says the
> remf version is faster, but I found it to be around four times slower
> for some data and insignificantly different otherwise.
pushed (once again blindly beliving the patch author).
but this time i also pushed a test at lest that tests for the removal
of duplicate keys.
sorry for blindly pushing the previous change.
--
• attila lendvai
• PGP: 963F 5D5F 45C7 DFCD 0A39
--
“Disregard for the preferences and interests of individuals alive
today in order to pursue some distant social goal that their rulers
have claimed is their duty to promote has been a common cause of
misery for people throughout the ages.”
— Isaiah Berlin (1909–1997)
_______________________________________________
Alexandria-devel mailing list
Alexandria-devel@common-lisp.net
http://common-lisp.net/cgi-bin/mailman/listinfo/alexandria-devel
