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

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

[debbugs-tracker] bug#15580: closed ([PATCH] 24.3.50; subword-capitalize


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#15580: closed ([PATCH] 24.3.50; subword-capitalize at the end of a buffer jumps to the beginning)
Date: Tue, 15 Oct 2013 12:35:03 +0000

Your message dated Mon, 14 Oct 2013 15:25:17 -0400
with message-id <address@hidden>
and subject line Re: bug#15580: [PATCH] 24.3.50; subword-capitalize at the end 
of a buffer jumps to the beginning
has caused the debbugs.gnu.org bug report #15580,
regarding [PATCH] 24.3.50; subword-capitalize at the end of a buffer jumps to 
the beginning
to be marked as done.

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


-- 
15580: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=15580
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: [PATCH] 24.3.50; subword-capitalize at the end of a buffer jumps to the beginning Date: Thu, 10 Oct 2013 00:01:22 -0700
Hi.

I'm observing subword-capitalize get confused when called near the end
of a buffer. Instead of doing nothing like capitalize-word does,
subword-capitalize jumps to around the beginning of the buffer and
capitalizes something there.

The cause is simple. subword-capitalize does

(re-search-forward "[[:alpha:]]" ....)

and uses the results of this search even if it failed. The attached
patch explicitly checks for search failure and exits the function in
that case.

Attachment: 0001-subword-capitalize-now-fails-gracefully-at-end-of-bu.patch
Description: Text Data


--- End Message ---
--- Begin Message --- Subject: Re: bug#15580: [PATCH] 24.3.50; subword-capitalize at the end of a buffer jumps to the beginning Date: Mon, 14 Oct 2013 15:25:17 -0400 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux)
> and uses the results of this search even if it failed. The attached
> patch explicitly checks for search failure and exits the function in
> that case.

Thanks.  I slightly simplified your patch (since re-search-forward
already signal search-failed if the 3rd arg is nil) and installed it.


        Stefan


PS: This is at least the second "tiny" patch you contributed, so you're
on course to hit the limit of what we can accept without
copyright paperwork.  If you'd like to sign this paperwork already to
avoid future delays, I can send you the needed forms.


--- End Message ---

reply via email to

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