emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#18388: closed (24.4.50; REGRESSION: incompatible c


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#18388: closed (24.4.50; REGRESSION: incompatible change to `replace-match-maybe-edit' (and no doc))
Date: Wed, 10 Feb 2016 00:48:02 +0000

Your message dated Wed, 10 Feb 2016 02:47:06 +0200
with message-id <address@hidden>
and subject line Re: bug#18388: 24.4.50; REGRESSION: incompatible change to 
`replace-match-maybe-edit' (and no doc)
has caused the debbugs.gnu.org bug report #18388,
regarding 24.4.50; REGRESSION: incompatible change to 
`replace-match-maybe-edit' (and no doc)
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
18388: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=18388
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 24.4.50; REGRESSION: incompatible change to `replace-match-maybe-edit' (and no doc) Date: Tue, 2 Sep 2014 06:28:45 -0700 (PDT)
You added argument BACKWARD to this function, and you did not add it as
an &optional parameter.  This breaks all existing code that uses the
function.  The parameter should be &optional.

What's more, you did not even mention the new arg in the doc string.


In GNU Emacs 24.4.50.1 (i686-pc-mingw32)
 of 2014-08-15 on LEG570
Bzr revision: 117706 address@hidden
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'



--- End Message ---
--- Begin Message --- Subject: Re: bug#18388: 24.4.50; REGRESSION: incompatible change to `replace-match-maybe-edit' (and no doc) Date: Wed, 10 Feb 2016 02:47:06 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.90 (x86_64-pc-linux-gnu)
>> >> You added argument BACKWARD to this function, and you did not add it as
>> >> an &optional parameter.  This breaks all existing code that uses the
>> >> function.  The parameter should be &optional.
>> >>
>> >> What's more, you did not even mention the new arg in the doc string.
>> >
>> > ping
>>
>> I wonder what code and how might use this internal function?
>
> Does it matter?  Could you please make the parameter optional
> (and document it)?

Thanks for your patience.  Now fixed.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]