[debbugs-tracker] bug#681: closed (Wrong C++ indentation in batch mode)

[GNU bug Tracking System]

Your message dated Tue, 07 Jan 2014 11:26:13 +0100
with message-id <address@hidden>
and subject line Re: bug#681: Wrong C++ indentation in batch mode
has caused the debbugs.gnu.org bug report #681,
regarding Wrong C++ indentation in batch mode
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
681: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=681
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: Wrong C++ indentation in batch mode Date: Sat, 9 Aug 2008 17:26:22 +0200 User-agent: KMail/1.10.0 (Linux/2.6.24-19-generic; KDE/4.1.0; i686; ; )

        Hello,

        The indentation in C++ mode does not seem to work properly in batch 
mode with 
the following code (indented in interactive mode):
"""
template <class A, class B>
Derived<A, B>::Derived():
  Base<A,
       B>() // Problem: wrong indentation in batch mode.
{
}
"""

        Now, define the Lisp function:
(defun cpp_indent ()
  (c-set-style "gnu")
  (c++-mode)
  (indent-region (point-min) (point-max) ())
  (save-buffer)
  )
        And apply it the previous function. It will result in:
"""
template <class A, class B>
Derived<A, B>::Derived():
  Base<A,
  B>() // Problem: wrong indentation in batch mode.
{
}
"""
        which is inconsistent with the interactive and is not, I presume, the 
targeted result.
        I found the problem in Emacs 23.0.60.1 (emacs snapshot in Kubuntu 8.04) 
and 
in Emacs 22.1.3. It does not appear in Emacs 21.4.1 (Debian Etch).


        Just an unrelated note about the indentation. I found a difference 
between the 
two latest versions I mentioned and the version 21.4.1. This code (indented 
with Emacs 22+):
"""
template <class A, class B>
Derived0<A, B>::Derived0():
  Base<A, B>
  ()
{
}
"""
        is indented by Emacs 21.4.1 as
"""
template <class A, class B>
Derived0<A, B>::Derived0():
  Base<A, B>
()
{
}
"""
        I like the new indentation better, and I suppose it was an improvement 
introduced in the latest versions. I mention it, just in case...


        I attach three files:
- test.cxx: a file to be indented (but already well indented);
- test-wrong_indentation.cxx: test.cxx after indentation in batch mode with 
Emacs 22+;
- cpp_indent.lisp: the indentation function. Used in: "emacs -batch test.cxx -
l cpp_indent.lisp -f cpp_indent".


        Thank you for your great work,
        Vivien Mallet.

test-wrong_indentation.cxx
Description: Text Data

test.cxx
Description: Text Data

cpp_indent.lisp
Description: Text document

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#681: Wrong C++ indentation in batch mode
>
>
>
>
> Date: 
>
> Tue, 07 Jan 2014 11:26:13 +0100
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux)
>
>
>
>
> Hi Vivien,
>
> Vivien Mallet <address@hidden> writes:
>
> >       The indentation in C++ mode does not seem to work properly in batch 
> > mode with 
> > the following code (indented in interactive mode):
> > """
> > template <class A, class B>
> > Derived<A, B>::Derived():
> >   Base<A,
> >        B>() // Problem: wrong indentation in batch mode.
> > {
> > }
> > """
> >
> >       Now, define the Lisp function:
> > (defun cpp_indent ()
> >   (c-set-style "gnu")
> >   (c++-mode)
> >   (indent-region (point-min) (point-max) ())
> >   (save-buffer)
> >   )
> >       And apply it the previous function. It will result in:
> > """
> > template <class A, class B>
> > Derived<A, B>::Derived():
> >   Base<A,
> >   B>() // Problem: wrong indentation in batch mode.
> > {
> > }
> > """
> >       which is inconsistent with the interactive and is not, I presume, the 
> > targeted result.
>
> I can't reproduce this.  I get the correct indentation.
>
> I'm closing this bug report, feel free to reopen it if needed.
>
> -- 
>  Bastien
>
>
> --- End Message ---