Eduardo Cavazos <[EMAIL PROTECTED]>
writes:
> On Saturday 15 November 2008 20:40:22 Jose A. Ortega Ruiz wrote:
>
>> Hi! This should complete, for now, the font lock improvements to
>> factor-mode (see below). Next thing, the tabs stuff.
>
> Awesome! Pushed to mainline.
>
Thanks!
>> BTW, i was wondering whether you find my requesting pulls in small
>> patches like this annoying: would you prefer bigger and fewer
>> changesets?
>
> Not annoying at all. All the improvements are welcome! I'm just
> glad 'factor.el' is in good hands and is getting some needed attention.
>
I'm very glad to help. I've sent another pull request with the
indentation stuff in a separate email. I'll come up with more little
improvements and cleanups to factor.el, but before i'd like to ask if
you'd be comfortable with a couple of cosmetic changes:
- I've been using the convention of using 'factor--foo' instead of
'factor-foo' for internal names. I'd like to homogenize factor.el
naming: what convention do you prefer?
- factor.el uses banners like this:
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
;; section
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
to separate sections. I find it more useful (and aesthetically
pleasant) using ^L (that's C-qC-l) for quick navigation:
^L
;;; section
No big deal, but if you don't feel strongly about it, i'd change
that.
Cheers,
jao
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Factor-talk mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/factor-talk
