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.

[Ted Zlatanov]

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.

Ted