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

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

[debbugs-tracker] bug#21501: closed (new Emacs functions for capitalizin


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#21501: closed (new Emacs functions for capitalizing text intelligently)
Date: Sat, 26 Sep 2015 08:14:01 +0000

Your message dated Sat, 26 Sep 2015 11:13:21 +0300
with message-id <address@hidden>
and subject line Re: bug#21501: new Emacs functions for capitalizing text 
intelligently
has caused the debbugs.gnu.org bug report #21501,
regarding new Emacs functions for capitalizing text intelligently
to be marked as done.

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


-- 
21501: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=21501
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: new Emacs functions for capitalizing text intelligently Date: Thu, 17 Sep 2015 00:32:32 -0400
I'm submitting some additional -dwim functions to that change the capitalization of text the way the user probably wants.

These work similarly to comment-dwim, or count-words: they act on the region if and only if it's active (more specifically, if (use-region-p) is true), and otherwise act on the word starting at point. Either way, they use existing functions in Emacs: e.g. #'upcase-dwim delegates either to #'upcase-region or #'upcase-word.

There are three functions: one for each of capitalizing text, upcasing text, and lowercasing text. The docstrings are based on the docstring for comment-dwim.

In my init file, I've changed the default mappings of M-u, M-l, and M-c to these new functions, and would support changing Emacs's defaults to them. However, I am led to believe that changing defaults isn't preferred, so this patch doesn't contain any of that. Even without changing defaults, I believe these functions are worth adding to Emacs.

The capitalization-related functions I'm using are all defined in C, so I can't put these new functions alongside them. I put the changes in simple.el, as that file's described as "A grab-bag of basic Emacs commands not specifically related to some major mode or to file-handling.". I'm happy to move the functions and create a new diff if there's a more appropriate file for them to live in.

Changelog entry:

* simple.el: Add functions for capitalizing text intelligently.
(capitalize-dwim): New function.
(upcase-dwim): New function.
(downcase-dwim): New function.

The diff is attached.

-Zachary Kanfer

Attachment: 0001-Add-functions-for-capitalizing-text-intelligently.patch
Description: Text Data


--- End Message ---
--- Begin Message --- Subject: Re: bug#21501: new Emacs functions for capitalizing text intelligently Date: Sat, 26 Sep 2015 11:13:21 +0300
> From: Zachary Kanfer <address@hidden>
> Date: Mon, 21 Sep 2015 14:52:01 -0400
> Cc: address@hidden
> 
> Here's an updated patch with Eli's comments addressed. Now, these new commands
> seem to have all the functionality of the existing *-region and *-word
> commands.

Thanks, I pushed them.

Note that with this contribution you've exhausted your limit of code
submissions that can be accepted without legal paperwork, so if you'd
like to continue your contribution (we hope you do), you will have to
do that paperwork.

Also, please in the future always include with your patches a log
message in the format described in CONTRIBUTE.

Thanks.


--- End Message ---

reply via email to

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