emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: add-change-log-entry


From: martin rudalics
Subject: Re: add-change-log-entry
Date: Wed, 18 Jul 2007 16:32:47 +0200
User-agent: Mozilla Thunderbird 1.0 (Windows/20041206)

> you forgot the to disable removing newline before "^T*":

I didn't even care.

>    fix bug #[ 1607416 ]: back_trace is not initialized when it is a register
>    * spvw.d (back_trace): do not init to NULL at the declaration, instead...
>    (init_memory): init back_trace to NULL here
>
> M-q ===>
>
>    fix bug #[ 1607416 ]: back_trace is not initialized when it is a
>    register * spvw.d (back_trace): do not init to NULL at the
>    declaration, instead...
>    (init_memory): init back_trace to NULL here
>
> (note "* spvw.d" in the middle of a line).

This is a different issue.  Please look at the attached patch and tell
me whether it works and/or breaks anything else.

> also, when the function (or file) list is too long, M-q should preserve
> the standard ChangeLog formatting:
>
>    * foo (bar, baz, zot): quux
>
> ===>
>
>    * foo (bar, baz)
>    (zot); quux
>
> instead of
>
>    * foo (bar, baz,
>    zot); quux
>
> (for the sake of font-lock which can fontify the former but not the latter)

I could make ", " non-breakable but that wouldn't help much.  Has this
or a similar problem already been resolved somewhere else?  We'd need a
similar solution for `change-log-indent'.
*** add-log.el  Wed Jul 18 10:49:50 2007
--- add-log.el  Wed Jul 18 16:18:00 2007
***************
*** 732,738 ****
    (interactive "P")
    (let ((end (progn (forward-paragraph) (point)))
        (beg (progn (backward-paragraph) (point)))
!       (paragraph-start (concat paragraph-start "\\|\\s *\\s(")))
      (fill-region beg end justify)
      t))
  
--- 732,739 ----
    (interactive "P")
    (let ((end (progn (forward-paragraph) (point)))
        (beg (progn (backward-paragraph) (point)))
!       (paragraph-start (concat paragraph-start "\\|\\s *\\s("))
!       (paragraph-separate (concat paragraph-separate "\\|\\s *\\*")))
      (fill-region beg end justify)
      t))
  

reply via email to

[Prev in Thread] Current Thread [Next in Thread]