Return-Path: X-Original-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id DD0029270 for ; Tue, 15 May 2012 21:14:29 +0000 (UTC) Received: (qmail 20891 invoked by uid 500); 15 May 2012 21:14:29 -0000 Delivered-To: apmail-incubator-callback-dev-archive@incubator.apache.org Received: (qmail 20862 invoked by uid 500); 15 May 2012 21:14:29 -0000 Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: callback-dev@incubator.apache.org Delivered-To: mailing list callback-dev@incubator.apache.org Received: (qmail 20854 invoked by uid 99); 15 May 2012 21:14:29 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 15 May 2012 21:14:29 +0000 X-ASF-Spam-Status: No, hits=-1.3 required=5.0 tests=FRT_ADOBE2,RCVD_IN_DNSWL_MED,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of fil@adobe.com designates 64.18.1.185 as permitted sender) Received: from [64.18.1.185] (HELO exprod6og103.obsmtp.com) (64.18.1.185) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 15 May 2012 21:14:24 +0000 Received: from outbound-smtp-2.corp.adobe.com ([193.104.215.16]) by exprod6ob103.postini.com ([64.18.5.12]) with SMTP ID DSNKT7LHGxfYtBgZ7gdqvbMRGG+nenrbbkh6@postini.com; Tue, 15 May 2012 14:14:04 PDT Received: from inner-relay-4.eur.adobe.com (inner-relay-4b [10.128.4.237]) by outbound-smtp-2.corp.adobe.com (8.12.10/8.12.10) with ESMTP id q4FLE2If015992 for ; Tue, 15 May 2012 14:14:02 -0700 (PDT) Received: from nahub01.corp.adobe.com (nahub01.corp.adobe.com [10.8.189.97]) by inner-relay-4.eur.adobe.com (8.12.10/8.12.9) with ESMTP id q4FL5RaR017867 for ; Tue, 15 May 2012 14:13:59 -0700 (PDT) Received: from nambxv01a.corp.adobe.com ([10.8.189.95]) by nahub01.corp.adobe.com ([10.8.189.97]) with mapi; Tue, 15 May 2012 14:13:42 -0700 From: Filip Maj To: "callback-dev@incubator.apache.org" Date: Tue, 15 May 2012 14:17:13 -0700 Subject: Re: Java Style Thread-Topic: Java Style Thread-Index: Ac0y35qwixjmUnoaQ8mt6qq2RX/7kA== Message-ID: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Microsoft-MacOutlook/14.2.2.120421 acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Virus-Checked: Checked by ClamAV on apache.org OK will add that to the style guide then. Please adjust Eclipse accordingly, everyone! On 5/15/12 2:12 PM, "Bryce Curtis" wrote: >+1 on 4 spaces, no tabs. > >On Tue, May 15, 2012 at 4:07 PM, Michael Brooks > wrote: >> +1 on 4 spaces in Java. >> >> On Tue, May 15, 2012 at 1:55 PM, Joe Bowser wrote: >> >>> +1 on spaces. It should be noted that this isn't the default in >>>Eclipse. >>> :( >>> >>> On Tue, May 15, 2012 at 1:52 PM, Drew Walters >>>wrote: >>> >>> > Should note, all the BlackBerry Java code is 4 space indentation >>> > except for maybe a few spots where someone else edited the file. >>> > >>> > On Tue, May 15, 2012 at 3:50 PM, Drew Walters >>> wrote: >>> > > +1 spaces >>> > > >>> > > On Tue, May 15, 2012 at 3:48 PM, Shazron wrote: >>> > >> Spaces. >>> > >> I've an issue filed against iOS regarding this: >>> > >> https://issues.apache.org/jira/browse/CB-625 >>> > >> >>> > >> >>> > >> On Tue, May 15, 2012 at 1:49 PM, Filip Maj wrote: >>> > >>> I suppose I should post my refs too >>> > >>> >>> > >>> [1] http://wiki.apache.org/cordova/StyleGuide >>> > >>> [2] >>> > >>> >>> > >>>=20 >>>http://www.oracle.com/technetwork/java/javase/documentation/codeconventi >>>ons >>> > >>> -136091.html#262 >>> > >>> >>> > >>> On 5/15/12 1:48 PM, "Filip Maj" wrote: >>> > >>> >>> > >>>>Let's see how this one pans out ;) >>> > >>>> >>> > >>>>This came up during the code review of the CordovaWebView stuff >>>for >>> > >>>>Android. Our style guide [1] specifies to follow the Eclipse >>>Coding >>> > >>>>Conventions, which point to standards set out by Sun, which of >>>course >>> > now >>> > >>>>points to an Oracle web site. >>> > >>>> >>> > >>>>The Sun standards say [2] about indentation: >>> > >>>> >>> > >>>>---- >>> > >>>> >>> > >>>>4 - Indentation >>> > >>>> >>> > >>>>Four spaces should be used as the unit of indentation. The exact >>> > >>>>construction of the indentation (spaces vs. tabs) is unspecified. >>> Tabs >>> > >>>>must be set exactly every 8 spaces (not 4). >>> > >>>> >>> > >>>>---- >>> > >>>> >>> > >>>> >>> > >>>>So my question is: tabs or spaces in Java code? >>> > >>>> >>> > >>>>My vote: spaces. >>> > >>>> >>> > >>>>Let your voice be heard. >>> > >>>> >>> > >>> >>> > >>>