X-Authentication-Warning: delorie.com: mail set sender to geda-user-bounces using -f X-Recipient: geda-user AT delorie DOT com X-Original-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=t7dSRICHLmf6BOIWfpi/2Bnywunt3TacXvcWm3GbGTg=; b=Hr7yTe/goh4neZ2av6eoKNjIxKXus0lC+cKKUsQTtXeadQPcJnj+vH00vSSf6NRFLZ MloGsgdbOQ6a/mpKgWSRrhMTeZLH1TrrlDaqt+gJaR41FFvVAhFh2hscHuTOfISKUCsX Jgbc2SbWo6JKAmwvWKf8iZckbUfttUF1YGw54tcvfKtUxICbBTUVXsyZngmyR9sjQOpJ G7y5wfknSIouSUdXcAJclH6jbYu9LObn4aHIANDZjwNVUziisgrojVesvbwzC9/ZTKfx 1Q3P9SdqC1KT7OEu1LxS6HG//KArtNR+sggpil//lCeqqV11NRqz+q4nBwPwC37zsZ+C et4g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:content-type; bh=t7dSRICHLmf6BOIWfpi/2Bnywunt3TacXvcWm3GbGTg=; b=DGwtFPS7BHhlpgG1Au8ikM0sP+IqZAjFxHucBXmfj5atq4iyrEgsrszsAFcm2Kn8k/ vfCzM9/Zu8nDmNnT16z4pSaAuVVwNaff7ywXuT+CEPdXanEjPlsWjpPlJ4QXVVBUvmgR cmqw3lMMv/B+pY2oEGr6RL1lLdk77xTkwTfioehdmdjr7/LgZBk2pUV7GA28Ttj7w8Em 5KSZH4zM/LknWsIUhNealGc6LTbQsCwWne2U14M0TD80N8D8R9wOyjmbHi1usHoujp5n CPvwNpc8EfYEULPsfmWSmh+N7BoMWTg85OKCE8bzrjtAPqrcUIFwZX8aYwTKOEm+KLPN l78A== X-Gm-Message-State: AG10YOQZ3s7RDGrfcHOQbauWM+pUAzgSGR3Cw3QhLwJ1991aVpj/2aCE3y0lpOYhdE/4JXpf86QsUVa3F/7PEg== MIME-Version: 1.0 X-Received: by 10.28.48.131 with SMTP id w125mr9091748wmw.18.1453066476413; Sun, 17 Jan 2016 13:34:36 -0800 (PST) In-Reply-To: <569BCC77.60700@iee.org> References: <569BCA7F DOT 70903 AT prochac DOT sk> <569BCC77 DOT 60700 AT iee DOT org> Date: Sun, 17 Jan 2016 12:34:36 -0900 Message-ID: Subject: Re: [geda-user] cleaner version of fixes to file format plugin From: "Britton Kerin (britton DOT kerin AT gmail DOT com) [via geda-user AT delorie DOT com]" To: geda-user AT delorie DOT com Content-Type: multipart/alternative; boundary=001a114242a4296dac05298e674e Reply-To: geda-user AT delorie DOT com Errors-To: nobody AT delorie DOT com X-Mailing-List: geda-user AT delorie DOT com X-Unsubscribes-To: listserv AT delorie DOT com Precedence: bulk --001a114242a4296dac05298e674e Content-Type: text/plain; charset=UTF-8 On Sun, Jan 17, 2016 at 8:16 AM, M. J. Everitt (m DOT j DOT everitt AT iee DOT org) [via geda-user AT delorie DOT com] wrote: > > On 17/01/16 17:08, Milan Prochac (milan AT prochac DOT sk) [via > geda-user AT delorie DOT com] wrote: > > On 17. 1. 2016 13:24, Peter Clifton (petercjclifton AT googlemail DOT com) > > [via geda-user AT delorie DOT com] wrote: > >> No // comments please. > >> > > > > Well, but they seems to be quite common: > > > > $ git status > > # On branch master > > nothing to commit, working directory clean > > $ grep -R // * | grep // | wc -l > > 527 > > > > Milan > > > Better to have some QA scripts that pick these things out, than use > compiler options. Even better still, use commit hooks to run these > scripts so that you cannot commit without fixing any issues. > Even better would be to ignore them, since they aren't really issues. It's window dressing, you might put it on after everything else is perfect, which is not the case here. Putting it on fixes nothing real. A number of devels don't actually like the current preferred style, but want to continue using it perhaps out of respect for the original devels. Who aren't even around any more. Indentation matters, local consistency matters, // vs /* and which line braces go on don't. Britton --001a114242a4296dac05298e674e Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

On Sun, Jan 17, 2016 at 8:16 AM, M. J. Everitt (m DOT j DOT everitt AT iee DOT org) [via geda-user AT delorie DOT com] <geda-user AT delorie DOT com><= /span> wrote:

On 17/01/16 17:08, Milan Prochac (milan= @prochac.sk) [via

geda-user AT d= elorie.com] wrote:
> On 17. 1. 2016 13:24, Peter Clifton (petercjclifton AT googlemail DOT com)
> [via geda-user AT delorie DOT com] wrote:
>> No // comments please.
>>
>
> Well, but they seems to be quite common:
>
> $ git status
> # On branch master
> nothing to commit, working directory clean
> $ grep -R // * | grep // | wc -l
> 527
>
> Milan
>

Better to have some QA scripts that pick these things out, than= use
compiler options. Even better still, use commit hooks to run these
scripts so that you cannot commit without fixing any issues.

Even better would be to ignore them, since= they aren't really issues.=C2=A0 It's window dressing, you might p= ut it on after everything else is perfect, which is not the case here.=C2= =A0 Putting it on fixes nothing real.

A number of devels don't actually like the current preferred = style, but want to continue using it perhaps out of respect for the origina= l devels.=C2=A0 Who aren't even around any more.
<= br>
Indentation matters, local consistency matters, //= vs /* and which line braces go on don't.

Britton