This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [doc RFA] Clean up coding standards, add python coding standards.
- From: Eli Zaretskii <eliz at gnu dot org>
- To: dje at google dot com (Doug Evans)
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 19 Oct 2010 20:44:36 +0200
- Subject: Re: [doc RFA] Clean up coding standards, add python coding standards.
- References: <20101019155311.038842461AD@ruffy.mtv.corp.google.com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Tue, 19 Oct 2010 08:53:10 -0700 (PDT)
> From: dje@google.com (Doug Evans)
>
> I think it is wrong that a discussion of cleanups appears in the same
> chapter as coding standards. Ditto for a few other subsections
> in the existing Coding chapter.
>
> This patch creates a new Coding Standards chapter to put such things in,
> and I've added a section on Python coding standards.
Thanks.
> I'm not happy with the name "Miscellaneous Coding Guidelines" but it
> works well enough for me. Feel free to suggest an alternatives.
"Misc Guidelines"? In general, node names should be short, because
they are displayed on the mode line and header line of the Info
readers, where space is at premium. (Section names, OTOH, can be
longer.)
> Ok to check in?
Could you tell me if you made any changes in the "Internal Error
Handling" part, or just moved it without any changes? If you just
moved it, I won't need to review that part.
> -* Coding::
> +* Miscellaneous Coding Guidelines::
> +* Coding Standards::
I think "Coding Standards" should be before the new "Misc" section.
> +@value{GDBN} follows the published @code{Python} coding standards in
> +@code{PEP008}.
> +See @uref{http://www.python.org/dev/peps/pep-0008/, PEP008}.
This is suboptimal usage of @uref. I suggest this instead:
@value{GDBN} follows the published @code{Python} coding standards in
@uref{http://www.python.org/dev/peps/pep-0008/, @code{PEP008}}.
> +In addition, the Google standards are also followed where they do not
> +conflict with @code{PEP008}.
> +See @uref{http://google-styleguide.googlecode.com/svn/trunk/pyguide.html,
> +Google Python Style Guide}.
Same here:
In addition, the guidelines in the
@uref{http://google-styleguide.googlecode.com/svn/trunk/pyguide.html,
Google Python Style Guide} are also followed where they do not
conflict with @code{PEP008}.
Okay with those changes (assuming that the "Internal Error Handling"
stuff was moved without any change).
Thanks for taking care of this.