public inbox for frysk@sourceware.org
 help / color / mirror / Atom feed
From: Mark Wielaard <mark@klomp.org>
To: Andrew Cagney <cagney@redhat.com>
Cc: frysk@sourceware.org
Subject: Re: adopting OpenJDK's coding standard[?]?
Date: Thu, 14 Jun 2007 11:57:00 -0000	[thread overview]
Message-ID: <1181815871.4474.34.camel@dijkstra.wildebeest.org> (raw)
In-Reply-To: <466FEC94.4000808@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 1653 bytes --]

Hi,

On Wed, 2007-06-13 at 09:09 -0400, Andrew Cagney wrote:
> Since OpenJDK is going to subsume CLASSPATH as the Free Java library, 
> and OpenJDK follows Sun/Java coding conventions (sort of mostly :-), I 
> can't think of a reason for continuing with the GNU format.  In 
> addition, its the format that EMACS already follows; and there should be 
> good indentation tools around.  The only -ve is that Eclipse might still 
> need to be, er, "tweaked".
> 
> Topic for todays meeting.

As I said in the meeting, but I didn't see the minutes yet, I think
changing coding conventions is a bad idea. It is basically a bikeshed
discussion waiting to happen [1] and everybody can come up with good
arguments for or against any style. We have a Frysk style now and I
would like to just stick with it instead of randomly changing it every
so often.

Just to insert my own bikeshed arguments:

- I like the project to have one consistent style. Switching between
styles just because one part is written in C, C++ or Java doesn't make
sense to me.
- We aren't based on OpenJDK, we use gcj and will for a time unless
someone starts adding CNI support to OpenJDK or rewrite all our current
wrappers and native code to support JNI (which will require analysis of
performance impacts again since JNI and CNI have very different
performance characteristics). GCJ obviously also uses GNU style like we
do now.
- And finally I like GNU style a lot better because the start and end of
code blocks are so clear to find, something the proposed Sun coding
style guide completely misses.

Cheers,

Mark

[1] http://www.bikeshed.com/

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

  parent reply	other threads:[~2007-06-14 10:11 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-13 13:12 Andrew Cagney
2007-06-14  9:16 ` Nurdin Premji
2007-06-14 15:17   ` Adam Jocksch
2007-06-14 11:57 ` Mark Wielaard [this message]
2007-06-14 13:59   ` Andrew Cagney

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

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

  git send-email \
    --in-reply-to=1181815871.4474.34.camel@dijkstra.wildebeest.org \
    --to=mark@klomp.org \
    --cc=cagney@redhat.com \
    --cc=frysk@sourceware.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.
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).