unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Andrew Hyatt <ahyatt@gmail.com>
To: Alan Mackenzie <acm@muc.de>
Cc: Bastien <bzg@altern.org>, Vivien Mallet <Vivien.Mallet@inria.fr>,
	681@debbugs.gnu.org
Subject: bug#681: Wrong C++ indentation in batch mode
Date: Mon, 25 Jan 2016 23:46:23 -0500	[thread overview]
Message-ID: <m260yh3q3k.fsf@gmail.com> (raw)
In-Reply-To: <20160104231642.GA13716@acm.fritz.box> (Alan Mackenzie's message of "Mon, 4 Jan 2016 23:16:42 +0000")

Alan Mackenzie <acm@muc.de> writes:

> Hello Vivien.
>
> On Mon, Dec 28, 2015 at 08:18:02PM -0500, Andrew Hyatt wrote:
>
>> Bastien <bzg@altern.org> writes:
>
>> > Hi Vivien,
>
>> > Vivien Mallet <Vivien.Mallet@inria.fr> writes:
>
>> >> Now, launch from command line:
>> >> \emacs -Q test.cpp --batch --eval="(defun cpp_indent () (c-set-style \"gnu\") (c++-mode) (indent-region (point-min) (point-max) ()) (save-buffer))" --eval="(cpp_indent)"
>
>> >> You should get:
>
>> >> template <class A, class B>
>> >> Derived<A, B>::Derived():
>> >>   Base<A,
>> >>   B>() // Problem: wrong indentation in batch mode.
>> >> {
>> >>   int i:
>> >> }
>
>> >> In addition, it seems that "c-set-style" is not honored. It may be
>> >> "bsd", "k&r", etc. without any difference.
>
>> > Mhh... yes, I see now.
>
>> > I reopend the bug and will let someone more familiar
>> > with the code fix it.
>
>> I'm not the one more familiar, but I just want to record that this still
>> reproduces on emacs 25.
>
> I've committed what I hope is a fix to the emacs-25 branch at savannah.
> Sorry it's taken (a lot) longer than it should have done.
>
> Would you please tell me what version of Emacs you're currently using,
> and I will try to adapt the patch for that version.  Then I will ask you
> to try out the patch and let me know whether or not there are still
> errors.
>
> Then, hopefully, we can close this bug.

Seeing as how there's no response, if you've fixed this bug (thanks,
BTW!), perhaps we should close it now?





  reply	other threads:[~2016-01-26  4:46 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-09 15:26 bug#681: Wrong C++ indentation in batch mode Vivien Mallet
2008-08-10 16:42 ` OFFICE ZERO
     [not found]   ` <handler.681.B681.12183870793367.ackinfo@emacsbugs.donarmstrong.com>
2008-08-10 17:10     ` bug#681: Info received (bug#681: Wrong C++ indentation in batch mode) OFFICE ZERO
     [not found]       ` <handler.681.B681.121838828710091.ackinfo@emacsbugs.donarmstrong.com>
2008-08-11  1:17         ` bug#681: Info received (bug#681: Info received (bug#681: Wrong C++ indentation in batch mode)) OFFICE ZERO
2008-08-10 17:01 ` bug#681: Wrong C++ indentation in batch mode OFFICE ZERO
     [not found]   ` <handler.681.B681.12183876726365.ackinfo@emacsbugs.donarmstrong.com>
2008-08-11  1:17     ` bug#681: Info received (bug#681: Wrong C++ indentation in batch mode) OFFICE ZERO
2008-08-10 17:11 ` bug#681: Wrong C++ indentation in batch mode OFFICE ZERO
     [not found]   ` <handler.681.B681.121838833410165.ackinfo@emacsbugs.donarmstrong.com>
2008-08-11  1:16     ` bug#681: Info received (bug#681: Wrong C++ indentation in batch mode) OFFICE ZERO
2008-08-10 17:12 ` bug#681: Wrong C++ indentation in batch mode OFFICE ZERO
     [not found]   ` <handler.681.B681.121838833410168.ackinfo@emacsbugs.donarmstrong.com>
2008-08-11  1:07     ` bug#681: Info received (bug#681: Wrong C++ indentation in batch mode) OFFICE ZERO
2014-01-07 10:26 ` bug#681: Wrong C++ indentation in batch mode Bastien Guerry
2014-01-07 15:36   ` Vivien Mallet
2014-01-07 16:09     ` Bastien
2015-12-29  1:18       ` Andrew Hyatt
2015-12-29 22:02         ` Alan Mackenzie
2016-01-04 23:16         ` Alan Mackenzie
2016-01-26  4:46           ` Andrew Hyatt [this message]
2016-02-06 12:05           ` Vivien Mallet
2016-02-07 22:09             ` Alan Mackenzie
2016-02-07 22:10               ` Vivien Mallet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m260yh3q3k.fsf@gmail.com \
    --to=ahyatt@gmail.com \
    --cc=681@debbugs.gnu.org \
    --cc=Vivien.Mallet@inria.fr \
    --cc=acm@muc.de \
    --cc=bzg@altern.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).