[Alexandria-devel] [alexandria-devel] Implementation of DELETE-FROM-PLIST

[Attila Lendvai]

> 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)