[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#1948: confusion and bug in dabbrev.el
From: |
Lars Ingebrigtsen |
Subject: |
bug#1948: confusion and bug in dabbrev.el |
Date: |
Mon, 29 Feb 2016 15:18:43 +1100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) |
Alan Third <alan@idiocy.org> writes:
> I think I've got a fix for this.
>
> The problem is that dabbrev--substitute-expansion alters EXPANSION
> before using it in the substitution, but dabbrev-expand has no idea that
> it's done this, so it assumes the unaltered EXPANSION is still valid and
> saves it.
>
> The patch makes dabbrev--substitute-expansion return EXPANSION, and
> dabbrev-expand saves that version instead.
>
> I considered making a new function to remove the whitespace and using it
> in both dabbrev-expand and dabbrev--substitute-expansion, but returning
> the value actually used strikes me as safer, in case anything else is
> done to it.
If I understand the code correctly, this looks like a good fix to me.
Could you resubmit it with the test case you sent earlier as one patch?
Also I see that we have a copyright disclaimer on file for your work.
Is that sufficient to accept code into Emacs? (I'm asking the other
Emacs hackers here...)
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
- bug#1948: confusion and bug in dabbrev.el,
Lars Ingebrigtsen <=