GNU bug report logs - #14626
CC Mode 5.32.5 (C++/l); indentation problem involving override

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Alex <agrambot <at> gmail.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#14626: closed (CC Mode 5.32.5 (C++/l); indentation problem
 involving override)
Date: Mon, 22 Aug 2016 04:14:01 +0000

[Message part 1 (text/plain, inline)]

Your message dated Sun, 21 Aug 2016 22:13:16 -0600
with message-id <874m6dqw4j.fsf <at> gmail.com>
and subject line Re: bug#14626: CC Mode 5.32.5 (C++/l); indentation problem involving override
has caused the debbugs.gnu.org bug report #14626,
regarding CC Mode 5.32.5 (C++/l); indentation problem involving override
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
14626: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=14626
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Michael Welsh Duggan <mwd <at> md5i.com>
To: bug-gnu-emacs <at> gnu.org
Subject: CC Mode 5.32.5 (C++/l); indentation problem involving override
Date: Sat, 15 Jun 2013 12:06:33 -0400

Date: Sat, 15 Jun 2013 12:03:45 -0400
Message-ID: <87a9mrqs2m.fsf <at> maru.md5i.com>
User-Agent: Gnus/5.130006 (Ma Gnus v0.6) Emacs/24.3.50 (gnu/linux)

In c++-mode, the following class gets indented correctly:

class Foo
{
  Foo
  foo()
    const;
};

as does this:

class Foo
{
  Foo *
  foo()
    const;
};

and this:

class Foo
{
  Foo
  foo()
    const
    override;
};


But the following does not:

class Foo
{
  Foo *
  foo()
  const
  override;
};


Emacs  : GNU Emacs 24.3.50.1 (i686-pc-linux-gnu, X toolkit)
 of 2013-06-11 on maru
Package: CC Mode 5.32.5 (C++/l)
Buffer Style: gnu
c-emacs-features: (pps-extended-state col-0-paren posix-char-classes gen-string-delim gen-comment-delim syntax-properties 1-bit)

current state:
==============
-- 
Michael Welsh Duggan
(md5i <at> md5i.com)

[Message part 3 (message/rfc822, inline)]

From: Alex <agrambot <at> gmail.com>
To: Michael Welsh Duggan <mwd <at> md5i.com>
Cc: 14626-done <at> debbugs.gnu.org
Subject: Re: bug#14626: CC Mode 5.32.5 (C++/l);
 indentation problem involving override
Date: Sun, 21 Aug 2016 22:13:16 -0600

Michael Welsh Duggan <mwd <at> md5i.com> writes:

> Date: Sat, 15 Jun 2013 12:03:45 -0400
> Message-ID: <87a9mrqs2m.fsf <at> maru.md5i.com>
> User-Agent: Gnus/5.130006 (Ma Gnus v0.6) Emacs/24.3.50 (gnu/linux)
>
> In c++-mode, the following class gets indented correctly:
>
> class Foo
> {
>   Foo
>   foo()
>     const;
> };
>
> as does this:
>
> class Foo
> {
>   Foo *
>   foo()
>     const;
> };
>
> and this:
>
> class Foo
> {
>   Foo
>   foo()
>     const
>     override;
> };
>
>
> But the following does not:
>
> class Foo
> {
>   Foo *
>   foo()
>   const
>   override;
> };

This indents correctly since 24.4.

> It looks to me that "noexcept", "override", and "final" may need to be
> added to c-type-modifier-kwds.  Or maybe more appropriately in
> c-type-decl-suffix-key, since they can only appear at the end of a
> declaration.  A brief scan through cc-langs.el shows that there might
> need to be a few other new c++11 symbols added as well, such as
> "constexpr".   "decltype" might be particularly troublesome, as it is a
> type that includes an argument in parentheses.

> Please note that "noexcept", like "throw", can have a parenthetical
> expression after it.

It appears that these keywords (apart from decltype) were added in time
for 25.1. decltype appears to be in master.

I'm closing the bug due to the above. Please re-open if you disagree.

Previous Next

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