"Luc Teirlinck" <[EMAIL PROTECTED]> wrote: > It would be useful for a few experienced Emacs developers to look it > [longlines.el] over and make suggestions. > > I am not really the best person to make suggestions. It would be > better if longtime users did. Here are some remarks. > ... > After putting (load "~/longlines.elc") in my .emacs, I had to edit > longlines.el and put: (defvar longlines-mode nil) at the beginning to > avoid a warnings buffer cropping up each time I started Emacs, warning > about the free variable `longlines-mode'. Just (defvar longlines-mode) > was _not_ sufficient.
I can't reproduce this, and it doesn't make sense; longlines-mode is defined using define-minor-mode, which should do the variable definition properly. Could you look into this further? Maybe it is something in your .emacs. > If I understood correctly, we do not like code included with the Emacs > distribution to use `defadvice'. longlines.el uses defadvice for > `newline', `kill-region', `copy-region-as-kill', `yank' and `yank-pop'. Yes, this will have to be fixed before longlines.el can go in. _______________________________________________ Emacs-devel mailing list Emacs-devel@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-devel
