GNU bug report logs - #7185
24.0.50; c-fill-paragraph regression

Previous Next

Packages: emacs, cc-mode;

Reported by: John Yates <john <at> yates-sheets.org>

Date: Sun, 10 Oct 2010 18:24:02 UTC

Severity: normal

Found in version 24.0.50

Done: Alan Mackenzie <acm <at> muc.de>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: John Yates <john <at> yates-sheets.org>
Subject: bug#7185: closed (Re: bug#7185: 24.0.50; c-fill-paragraph regression)
Date: Sun, 31 Oct 2010 11:48:02 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#7185: 24.0.50; c-fill-paragraph regression

which was filed against the emacs,cc-mode package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 7185 <at> debbugs.gnu.org.

-- 
7185: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=7185
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Alan Mackenzie <acm <at> muc.de>
To: John Yates <john <at> yates-sheets.org>, 7185-done <at> debbugs.gnu.org
Subject: Re: bug#7185: 24.0.50; c-fill-paragraph regression
Date: Sun, 31 Oct 2010 12:04:43 +0000
Hi, John,

On Sun, Oct 10, 2010 at 11:53:15AM -0400, John Yates wrote:
> ==[ Before filling comment ]==
> 
> /*
>     >> abc
>     >> def
> */
> {
>     return 0;
> }

> ==[ Emacs 23.1.1, after filling (sub-optimal) ]==
> 
> /*
>     >> abc def */
> {
>     return 0;
> }

> ==[ Emacs 24.0.50.1, after filling (much worse) ]==
> 
> /*
>     >> abc def */ { return 0; }
> 

> The problem seems to associated with the fact the the comment close
> token (*/) is not preceded by an initial space (which, had it been
> present, would have cause the opening and closing asterisks to be
> aligned.

It was actually an off-by-1 error in `c-mask-paragraph'.  It's now fixed
(I hope :-).  I'd by grateful if you could try out the fix (now in the
bzr repository), just to make sure I haven't introduced a new bug.

Thanks for such a high quality bug report!
 
-- 
Alan Mackenzie (Nuremberg, Germany).

[Message part 3 (message/rfc822, inline)]
From: John Yates <john <at> yates-sheets.org>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.0.50; c-fill-paragraph regression
Date: Sun, 10 Oct 2010 11:53:15 -0400
==[ Before filling comment ]==

/*
    >> abc
    >> def
*/
{
    return 0;
}

==[ Emacs 23.1.1, after filling (sub-optimal) ]==

/*
    >> abc def */
{
    return 0;
}

==[ Emacs 24.0.50.1, after filling (much worse) ]==

/*
    >> abc def */ { return 0; }


The problem seems to associated with the fact the the comment close
token (*/) is not preceded by an initial space (which, had it been
present, would have cause the opening and closing asterisks to be
aligned.

==[ Before filling paragraph ]==

/*
    >> abc
    >> def
 */
{
    return 0;
}

==[ Either Emacs, after filling (optimal) ]==

/*
    >> abc def
 */
{
    return 0;
}


In GNU Emacs 24.0.50.1 (x86_64-pc-linux-gnu, GTK+ Version 2.20.1)
 of 2010-10-08 on elegiac, modified by Debian
 (emacs-snapshot package, version 1:20101008-1)
Windowing system distributor `The X.Org Foundation', version 11.0.10706000
configured using `configure  '--build' 'x86_64-linux-gnu' '--host'
'x86_64-linux-gnu' '--prefix=/usr' '--sharedstatedir=/var/lib'
'--libexecdir=/usr/lib' '--localstatedir=/var'
'--infodir=/usr/share/info' '--mandir=/usr/share/man' '--with-pop=yes'
'--enable-locallisppath=/etc/emacs-snapshot:/etc/emacs:/usr/local/share/emacs/24.0.50/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/24.0.50/site-lisp:/usr/share/emacs/site-lisp'
'--without-compress-info' '--with-x=yes' '--with-x-toolkit=gtk'
'--with-imagemagick=yes' 'build_alias=x86_64-linux-gnu'
'host_alias=x86_64-linux-gnu' 'CFLAGS=-DDEBIAN
-DSITELOAD_PURESIZE_EXTRA=5000 -g -O2' 'LDFLAGS=-g -Wl,--as-needed'
'CPPFLAGS=''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_US.utf8
  value of $XMODIFIERS: nil
  locale-coding-system: utf-8-unix
  default enable-multibyte-characters: t

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
<help-echo> <escape> x r e p o r t - <tab> <return
>

Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.

Load-path shadows:
/usr/share/emacs/24.0.50/site-lisp/debian-startup hides
/usr/share/emacs/site-lisp/debian-startup

Features:
(shadow sort gnus-util mail-extr message sendmail regexp-opt rfc822 mml
easymenu mml-sec mm-decode mm-bodies mm-encode mail-parse rfc2231
rfc2047 rfc2045 ietf-drums mm-util mail-prsvr mailabbrev mail-utils
gmm-utils mailheader emacsbug tooltip ediff-hook vc-hooks
lisp-float-type mwheel x-win x-dnd tool-bar dnd fontset image fringe
lisp-mode register page menu-bar rfn-eshadow timer select scroll-bar
mouse jit-lock font-lock syntax facemenu font-core frame cham georgian
utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao korean
japanese hebrew greek romanian slovak czech european ethiopic indian
cyrillic chinese case-table epa-hook jka-cmpr-hook help simple abbrev
loaddefs button minibuffer faces cus-face files text-properties overlay
md5 base64 format env code-pages mule custom widget
hashtable-print-readable backquote make-network-process dbusbind
dynamic-setting font-render-setting move-toolbar gtk x-toolkit x
multi-tty emacs)



This bug report was last modified 14 years and 209 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.