[Top][All Lists]

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

Re: /srv/bzr/emacs/trunk r112183: Move forward-whitespace, forward-symbo

From: Ted Zlatanov
Subject: Re: /srv/bzr/emacs/trunk r112183: Move forward-whitespace, forward-symbol, forward-same-syntax commands to subr.el. Use forward-symbol in supermode.el again.
Date: Sun, 31 Mar 2013 08:10:12 -0400
User-agent: Gnus/5.130006 (Ma Gnus v0.6) Emacs/24.3.50 (gnu/linux)

On Sun, 31 Mar 2013 14:06:38 +0800 Leo Liu <address@hidden> wrote: 

LL> On 2013-03-30 20:10 +0800, Ted Zlatanov wrote:
>> http://permalink.gmane.org/gmane.emacs.devel/158448
>> Stefan advised it for `forward-symbol' and after looking at
>> `thingatpt.el', I agreed with him.  The other two (`forward-same-syntax'
>> and `forward-whitespace') were very similar so I followed the same
>> logic:
>> 1) simple, generally useful, small
>> 2) use only basic Emacs commands and no commands from `thingatpt.el'
>> 3) seem consistent with the rest of `subr.el'
>> Perhaps we can have `motion.el' or something, but with the current
>> layout `subr.el' seems the best place.

LL> You already have a nice and broad name space to work with i.e. thingatpt
LL> so no need to make another one. thingatpt gives you forward and backward
LL> motions and other goodies.

LL> I think a better and cleaner way to do this is maybe clean up
LL> thingatpt.el. Now codes are scattered everywhere which is messier in my
LL> book.

I explained my reasons and you've restated yours.

I'll let the Emacs maintainers decide, since Stefan suggested it.  I'll
gladly do whatever they recommend.


reply via email to

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