CEDET

Hi,

the attached patched lets srecode-insert handle &rest dict-entries
according to their types, which is convenient when used in interactive
situations like this:

(define-abbrev-table 
  'c++-mode-abbrev-table 
  '(("incs"
     "" 
     (lambda ()
       (srecode-load-tables-for-mode major-mode)
       (srecode-insert "declaration:include" "SYSTEM" 't)))
[...]

Since the patch can possible break srecode-insert, I did not commit it
to CVS yet. I would appreciate comments.

Kind regards,
Jan

Re: [CEDET-devel] [PATCH]: srecode-insert.el: srecode-insert: handle &rest arguments based on types From: Eric M. Ludlam <eric@si...> - 2009-01-21 00:22 >>> Jan Moringen <jan.moringen@...> seems to think that: >Hi, > >the attached patched lets srecode-insert handle &rest dict-entries >according to their types, which is convenient when used in interactive >situations like this: > >(define-abbrev-table > 'c++-mode-abbrev-table > '(("incs" > "" > (lambda () > (srecode-load-tables-for-mode major-mode) > (srecode-insert "declaration:include" "SYSTEM" 't))) >[...] > >Since the patch can possible break srecode-insert, I did not commit it >to CVS yet. I would appreciate comments. [ ... ] I like your idea. Both the use of srecode in abbrev, and the extended use of dict-entries. My only caution is in the use of multiple-value-bind. Several cl-macs.el functions emit warnings from the byte-compiler about their use. There used to be push-back against using CL in Emacs programs. I don't know what the current maintainers think about this. Of course, many uses of cl features do go into CEDET. They are quite useful. I'm not familiar with that one though. Thanks Eric -- Eric Ludlam: eric@... Siege: http://www.siege-engine.com Emacs: http://cedet.sourceforge.net

Re: [CEDET-devel] [PATCH]: srecode-insert.el: srecode-insert: handle &rest arguments based on types From: Lennart Borgman <lennart.borgman@gm...> - 2009-01-21 00:28 On Wed, Jan 21, 2009 at 1:22 AM, Eric M. Ludlam <eric@...> wrote: > My only caution is in the use of multiple-value-bind. Several > cl-macs.el functions emit warnings from the byte-compiler about their > use. There used to be push-back against using CL in Emacs programs. > I don't know what the current maintainers think about this. See (info "(elisp) Coding Conventions")

Re: [CEDET-devel] [PATCH]: srecode-insert.el: srecode-insert: handle &rest arguments based on types From: Eric M. Ludlam <eric@si...> - 2009-01-21 02:47 >>> Lennart Borgman <lennart.borgman@...> seems to think that: >On Wed, Jan 21, 2009 at 1:22 AM, Eric M. Ludlam <eric@...> wrote: >> My only caution is in the use of multiple-value-bind. Several >> cl-macs.el functions emit warnings from the byte-compiler about their >> use. There used to be push-back against using CL in Emacs programs. >> I don't know what the current maintainers think about this. > >See > > (info "(elisp) Coding Conventions") Ah, so if this line: (eval-when-compile (require 'cl)) is added with the other require statements, the macro will expand, and things will be good. Thanks Eric -- Eric Ludlam: eric@... Siege: http://www.siege-engine.com Emacs: http://cedet.sourceforge.net

1 message has been excluded from this view by a project administrator.