Re: newbie hacker question: how to add eval-to-texinfo?

[jemarch@gnu.org]

    > I want to add a function called 'eval-to-texi' to emacs (i already
    > signed papers for that).
    
    The name does not make it very clear to me what it does.  Could you explain?

Oh, sorry. 

(defun eval-to-texi (form &optional tostring)
  "Evaluates FORM, dumping a @lisp texinfo environment with the
action.

If TOSTRING is t, then the @lisp environment is returned into a string,
rather than being inserted into the buffer."

  ...)

Examples:

@lisp
(car '(a b c))
      @result{} a
@end lisp

@lisp
(car (a b c))
      @error{} Symbol's function definition is void: a
@end lisp


    
    > - I suppose texinfo.el is the right way to add it. ¿It is right?.
    
    Depends on the patch.  But it's a very minor issue.  Most patches are not
    applied as is but are first sent back for a few minor changes: just send
    the patch and we'll tell you what needs to be changed, if anything.
    
    > - All functions in texinfo.el start with a 'texinfo-' prefix. May i to
    >   set that prefix to 'eval-to-texi' naming it 'texinfo-eval-to-texi'?
    
    Again, depends on the patch.

Ok.
    
    > - May i to include an autoload for 'eval-to-texi'?
    
    I don't see why not.
    
    > - To make the patch in unidiff format, would be right to generate it
    >   using 'cvs diff texinfo.el' on a fresh emacs cvs checkout? Or from
    >   the root directory: 'cvs diff lisp/textmodes/texinfo.el'
    
    Same difference.
     
    > - May i to add an entry to lisp/ChangeLog and send a patch for it too?
    
    Better to send the ChangeLog entry separately, but in a patch is
    also acceptable.  In most cases the patch is wrong (because the date
    needs to be changed, the file has changed in the mean time, etc..)
    so it has to be inserted by hand anyway.
    
    
Ok. Many thanks.

-- 
José E. Marchesi 

<jemarch@gnu.org>         http://www.gnu.org     GNU No es Unix!
<jemarch@es.gnu.org>      http://es.gnu.org      GNU España

.---------------.
|    .-[]--.    |
|   [] `>[]-|   |  BEE GNU/Hurd            http://bee.es.gnu.org
|    `-.[]<-'   |               
._______________.