Return-Path: X-Original-To: apmail-activemq-dev-archive@www.apache.org Delivered-To: apmail-activemq-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 9FCDD18BA0 for ; Tue, 4 Aug 2015 15:36:39 +0000 (UTC) Received: (qmail 11805 invoked by uid 500); 4 Aug 2015 15:36:39 -0000 Delivered-To: apmail-activemq-dev-archive@activemq.apache.org Received: (qmail 11760 invoked by uid 500); 4 Aug 2015 15:36:39 -0000 Mailing-List: contact dev-help@activemq.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@activemq.apache.org Delivered-To: mailing list dev@activemq.apache.org Received: (qmail 11748 invoked by uid 99); 4 Aug 2015 15:36:39 -0000 Received: from Unknown (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 04 Aug 2015 15:36:39 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id AE76319A3FA for ; Tue, 4 Aug 2015 15:36:38 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.9 X-Spam-Level: X-Spam-Status: No, score=0.9 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, KAM_LIVE=1, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=disabled Authentication-Results: spamd3-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com Received: from mx1-eu-west.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id KuooOOwvIiQl for ; Tue, 4 Aug 2015 15:36:26 +0000 (UTC) Received: from mail-pd0-f169.google.com (mail-pd0-f169.google.com [209.85.192.169]) by mx1-eu-west.apache.org (ASF Mail Server at mx1-eu-west.apache.org) with ESMTPS id 55FB820773 for ; Tue, 4 Aug 2015 15:36:26 +0000 (UTC) Received: by pdrg1 with SMTP id g1so5924402pdr.2 for ; Tue, 04 Aug 2015 08:35:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=6L2ycRPrBLN7+VuVnUMOwB6bRwgP83FAT7P0ntN8oPk=; b=nakjBV4+yIWnLUo97iHOcZiY8exSyp1vad71FlyO9cnHfejCobfaDrZt8eceVFQGwR 2G8LC5dJrpI1PWI2+xv9TbCiRFMwUm8ijWUP3J1OZq0RVUwJnlXR4VwxZgLQPuXSDy64 DcA+kCryfIZjtyDdzOd32+Olb1lGiQu9rXcgnelXhlkpu2jwSBYwP24yzJpAePU7bIlE zjgGbQNNBmXo1k6L/z+OXUrZW7bBfdnLJHl+Q5ocZdd9XDklmm8LPZScECJ/rAXk8LAW lSjMzj9gAwYEkqXDv7LomLgpK+ptryiCyWfw2bl/mOyEaYvKZWq1yMoyD1UgLWogOKuB suAw== MIME-Version: 1.0 X-Received: by 10.70.38.69 with SMTP id e5mr8930742pdk.27.1438702533774; Tue, 04 Aug 2015 08:35:33 -0700 (PDT) Received: by 10.70.30.129 with HTTP; Tue, 4 Aug 2015 08:35:33 -0700 (PDT) In-Reply-To: References: <55C0C70C.1000902@gmail.com> Date: Tue, 4 Aug 2015 11:35:33 -0400 Message-ID: Subject: Re: [DISCUSS] Artemis Coding Style From: Clebert Suconic To: "dev@activemq.apache.org" Content-Type: text/plain; charset=UTF-8 I think I will start with what we have now by just switching the curls to the same line.. Looking at our checkstyle they are not as strict as CXF (the one mentioned here). We can improve later if needed without any problems. On Tue, Aug 4, 2015 at 11:28 AM, Daniel Kulp wrote: > Lazy consensus is fine. Wait till tomorrow and if no one objects, just do it. > > Dan > > >> On Aug 4, 2015, at 11:24 AM, Clebert Suconic wrote: >> >> Is this a just do it thing, or we need a vote? >> >> On Tue, Aug 4, 2015 at 10:07 AM, Timothy Bish wrote: >>> On 08/04/2015 07:56 AM, Christopher Shannon wrote: >>>> So in general I'm not too picky with coding styles but I just started >>>> looking at the Artemis project a few days ago and something that stood out >>>> to me right away was the use of opening curly braces on a new line. >>>> >>>> Virtually ever Java code base I've seen is written in the style using the >>>> opening brace on the same line. (See >>>> http://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style) >>>> I think that in general it would be a good idea to match up to a style >>>> that most Java developers are used to working with if we want to get more >>>> of the community involved. >>>> >>>> I was wondering if anyone would have an issue with changing the style or >>>> what people's thoughts are about this potential change? >>>> >>> +1 >>> >>> I think following the javaguide style would make the code much more >>> approachable. >>> >>> -- >>> Tim Bish >>> Sr Software Engineer | RedHat Inc. >>> tim.bish@redhat.com | www.redhat.com >>> twitter: @tabish121 >>> blog: http://timbish.blogspot.com/ >>> >> >> >> >> -- >> Clebert Suconic > > -- > Daniel Kulp > dkulp@apache.org - http://dankulp.com/blog > Talend Community Coder - http://coders.talend.com > -- Clebert Suconic