<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#ffffff">
On 10/27/2009 02:10 PM, Manfred Bergmann wrote:
<blockquote cite="mid:7A29295E07F74B2E8A155DACDF7945F8@Crafter"
type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<meta name="GENERATOR" content="MSHTML 8.00.6001.18828">
<style></style>
<div> </div>
<blockquote
style="border-left: 2px solid rgb(0, 0, 0); padding-left: 5px; padding-right: 0px; margin-left: 5px; margin-right: 0px;">
<div
style="font-family: arial; font-style: normal; font-variant: normal; font-weight: normal; font-size: 10pt; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none;">-----
Original Message ----- </div>
<div
style="background: rgb(228, 228, 228) none repeat scroll 0% 0%; font-family: arial; font-style: normal; font-variant: normal; font-weight: normal; font-size: 10pt; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; -moz-background-clip: border; -moz-background-origin: padding; -moz-background-inline-policy: continuous;"><b>From:</b>
<a moz-do-not-send="true" title="joe@eireneh.com"
href="mailto:joe@eireneh.com">Joe Walker</a> </div>
<div
style="font-family: arial; font-style: normal; font-variant: normal; font-weight: normal; font-size: 10pt; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none;"><b>To:</b>
<a moz-do-not-send="true" title="jsword-devel@crosswire.org"
href="mailto:jsword-devel@crosswire.org">J-Sword Developers Mailing
List</a> </div>
<div
style="font-family: arial; font-style: normal; font-variant: normal; font-weight: normal; font-size: 10pt; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none;"><b>Sent:</b>
Tuesday, October 27, 2009 5:44 PM</div>
<div
style="font-family: arial; font-style: normal; font-variant: normal; font-weight: normal; font-size: 10pt; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none;"><b>Subject:</b>
Re: [jsword-devel] coding conventions</div>
<div><br>
</div>
<br>
<div class="gmail_quote">On Tue, Oct 27, 2009 at 1:09 PM, DM Smith <span
dir="ltr"><<a moz-do-not-send="true"
href="mailto:dmsmith@crosswire.org">dmsmith@crosswire.org</a>></span>
wrote:<br>
<blockquote
style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"
class="gmail_quote">
<div class="im"><br>
On Oct 27, 2009, at 8:32 AM, Manfred Bergmann wrote:<br>
<br>
<blockquote
style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"
class="gmail_quote">Hi have a question about conventions.<br>
<br>
Java coding guideline is to have the brackets not in a new line.<br>
I've seen both in JSword but is it to have new line?<br>
</blockquote>
<br>
</div>
The JSword guideline is to have every { and } on a new line. This
pre-existed my involvement.<br>
</blockquote>
<div><br>
It's my fault.<br>
My original thought was that { and } on new lines was more readable and
that the more compact version only made sense when we were restricted
to 80x24. I'm now of the opinion that the above might be true, however
uniformity is more important, so I'd now do that the more standard way.</div>
<div> </div>
</div>
<div class="gmail_quote">
<div>
<div><font face="Arial" size="2">I also found the { } in new lines
better readable in the past.</font></div>
<div><font face="Arial" size="2">Now however my sight about this
has changed. Brackets in new line pull too much the attention and
distract too much from the code which actually is the more inportant
thing.</font><br>
</div>
</div>
</div>
</blockquote>
</blockquote>
I'm up for a change on this one. We just need to settle on a good
convention. (Input wanted!)<br>
<br>
I've never been a fan of { being on it's own line, except when it
enhanced readability, as in:<br>
if (a &&<br>
b) {<br>
c;<br>
}<br>
Where a && b is really any compound condition spanning several
lines and c appears to line up with the list of conditions. In this
case I want to have something to set c apart from the conditions.<br>
This could either be the double indentation of the continuing
conditions:<br>
if (a &&<br>
b) {<br>
c;<br>
}<br>
Or of { being on the next line:<br>
if (a &&<br>
b)<br>
{<br>
c;<br>
}<br>
I think the extra indent makes a bit more sense and is easier to
enforce with checkstyle.<br>
<br>
Also, I think that for the sake of maintenance and adding new
expressions (e.g. debugging) that all single expressions are in a
block. That is, not:<br>
if (condition)<br>
expression;<br>
<br>
and that there are no one liners:<br>
if (condition) { expression; }<br>
<br>
I also prefer:<br>
} else {<br>
over:<br>
}<br>
else {<br>
<br>
Regarding methods, it gets pretty long to have:<br>
scope returnType methodName(parameterList) exceptionList {<br>
code;<br>
}<br>
I think this and class declarations are two places where { should be on
the next line. But I could be persuaded otherwise.<br>
<br>
If we change, I'd like to do it one package at a time and have
consistency within a package. The Eclipse formatter will make easy work
of it.<br>
<br>
Input?<br>
<br>
In Him,<br>
DM<br>
</body>
</html>