Re: [PATCH] bookmark.el of edit-utils 1.43 (completing-read usage error)
Adrian Aichner writes:
>>>>> "Hrvoje" == Hrvoje Niksic
<hniksic(a)iskon.hr> writes:
Hrvoje> I'm not sure that patch is right. The DEFAULT argument
Hrvoje> didn't exist in pre-21.2 XEmacs, and this patch leaves it
Hrvoje> in. That looks wrong.
Hi Hrvoje, Andreas, Yoshiki!
upon further examination I agree with Hrvoje. My quick patch is
wrong.
Doing it right (following Kyle's suggestion) would in this particular
case lead to awkward code bloat compared to the version before
Yoshiki's patch.
I think we should get used to using and seeing code like this.
Otherwise we'll never use new API features in the packages.