Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > comp.lang.python > #42766

Re: In defence of 80-char lines

Path csiph.com!usenet.pasdenom.info!aioe.org!news.stack.nl!newsfeed.xs4all.nl!newsfeed3.news.xs4all.nl!xs4all!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
Return-Path <joshua.landau.ws@gmail.com>
X-Original-To python-list@python.org
Delivered-To python-list@mail.python.org
X-Spam-Status OK 0.003
X-Spam-Evidence '*H*': 0.99; '*S*': 0.00; 'parameters': 0.04; 'argument': 0.05; 'insert': 0.05; 'skip:\xa0 30': 0.05; 'arguments': 0.09; 'cc:addr:python-list': 0.11; '&gt;&gt;': 0.16; 'cleaner': 0.16; 'prefered': 0.16; 'rarely': 0.16; 'readable': 0.16; 'tempted': 0.16; 'to:addr:python.list': 0.16; 'to:addr:tim.thechases.com': 0.16; 'to:name:tim chase': 0.16; 'unpacking': 0.16; 'wrote:': 0.18; 'skip:f 30': 0.19; 'cc:addr:python.org': 0.22; 'cc:2**1': 0.23; 'apply.': 0.24; 'typical': 0.24; '(or': 0.24; "i've": 0.25; '&gt;': 0.26; 'class.': 0.26; 'header:In-Reply-To:1': 0.27; 'point': 0.28; 'function': 0.29; 'properties': 0.29; 'tim': 0.29; '8bit%:3': 0.30; 'css': 0.30; 'ease': 0.30; 'message-id:@mail.gmail.com': 0.30; 'chase': 0.31; "we're": 0.32; "i'd": 0.34; 'skip:s 30': 0.35; 'something': 0.35; 'received:google.com': 0.35; 'add': 0.35; 'there': 0.35; 'keyword': 0.36; 'should': 0.36; 'skip:& 20': 0.39; '8bit%:6': 0.40; 'how': 0.40; 'most': 0.60; 'tracking': 0.61; 'more': 0.64; 'here': 0.66; 'reads': 0.68; ':).': 0.84; 'otten': 0.84; 'imagine': 0.93; '2013': 0.98
DKIM-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:mime-version:in-reply-to:references:from:date:message-id :subject:to:cc:content-type; bh=okM3k6F5vTHh1A2VZ1NXk+mR5k0qOuoA+LOwbkBh7XY=; b=eX9s2H/1s6nHT2R6qflklZl2Cq5y50z1PUMREAu3+EC8tBVv2CAFIwWYDGXaZ06B+b 5UvYdA2CSYMbuKbWJH7/CyHzcbBkY53w+p5TIqTrvn9PIIC3yYOf2Ws+UDlhf/MxMzmA hJeTUUsiFMIn801AEZue9QZkotI+mtCP+VbN+wQeLclMr8YsXv9qO/JX3PeIi/Q40Wvy JXPwN3rKFje2G9k4ttL3aT4blmb/WB0U4y7mhzxJSOYcKPTgYt+meJOjKy6zF4S6N5pt s8Maj21V7W3R5oq5Mf965WUKm5Z3SkU/5AL5Hn/9vL/TX+mBidwCw+lkEcogK7DNL4kW Sugg==
X-Received by 10.112.20.133 with SMTP id n5mr3932415lbe.113.1365095920217; Thu, 04 Apr 2013 10:18:40 -0700 (PDT)
MIME-Version 1.0
In-Reply-To <20130404060913.2e771293@bigbox.christie.dr>
References <515cd919$0$29966$c3e8da3$5496439d@news.astraweb.com> <c338b844-e9ce-46a7-9daf-20374372390e@googlegroups.com> <kjj7bk$22k$1@ger.gmane.org> <20130404060913.2e771293@bigbox.christie.dr>
From Joshua Landau <joshua.landau.ws@gmail.com>
Date Thu, 4 Apr 2013 18:18:00 +0100
Subject Re: In defence of 80-char lines
To Tim Chase <python.list@tim.thechases.com>
Content-Type multipart/alternative; boundary=14dae94732d368b96b04d98c29dd
Cc python-list <python-list@python.org>
X-BeenThere python-list@python.org
X-Mailman-Version 2.1.15
Precedence list
List-Id General discussion list for the Python programming language <python-list.python.org>
List-Unsubscribe <http://mail.python.org/mailman/options/python-list>, <mailto:python-list-request@python.org?subject=unsubscribe>
List-Archive <http://mail.python.org/pipermail/python-list/>
List-Post <mailto:python-list@python.org>
List-Help <mailto:python-list-request@python.org?subject=help>
List-Subscribe <http://mail.python.org/mailman/listinfo/python-list>, <mailto:python-list-request@python.org?subject=subscribe>
Newsgroups comp.lang.python
Message-ID <mailman.109.1365095929.3114.python-list@python.org> (permalink)
Lines 159
NNTP-Posting-Host 2001:888:2000:d::a6
X-Trace 1365095929 news.xs4all.nl 6985 [2001:888:2000:d::a6]:41126
X-Complaints-To abuse@xs4all.nl
Xref csiph.com comp.lang.python:42766

Show key headers only | View raw


[Multipart message — attachments visible in raw view] - view raw

On 4 April 2013 12:09, Tim Chase <python.list@tim.thechases.com> wrote:

> On 2013-04-04 08:43, Peter Otten wrote:
> > llanitedave wrote:
> >> self.mainLabel.SetFont(wx.Font(12, wx.DEFAULT, wx.NORMAL, wx.BOLD,
> faceName = "FreeSans"))
> >
> > I think I would prefer
> >
> > labelfont = wx.Font(
> >     pointSize=12,
> >     style=wx.DEFAULT,
> >     family=wx.NORMAL,
> >     weight=wx.BOLD,
> >     faceName="FreeSans")
> > self.mainLabel.SetFont(labelfont)
>
> +1
> The only change I'd make to this suggestion would be to add a
> semi-superfluous comma+newline after the last keyword argument too:
>
>  labelfont = wx.Font(
>      pointSize=12,
>      style=wx.DEFAULT,
>      family=wx.NORMAL,
>      weight=wx.BOLD,
>      faceName="FreeSans",
>      )
>

Since we're all showing opinions, I've always prefered the typical block
indentation:

labelfont = wx.Font(
    pointSize=12,
    style=wx.DEFAULT,
    family=wx.NORMAL,
    weight=wx.BOLD,
    faceName="FreeSans",
) # Not indented here

as

A(
    B(
        C,
        D,
        E,
    )
)

reads a lot cleaner than

A(
    B(
        C,
        D,
        E
        )
    )

which makes diffs cleaner when you need to insert something after
> faceName:
>
<DIFS SNIP>

That is a very good point :).

Additionally, if there are lots of keyword parameters like this, I'd
> be tempted to keep them in sorted order for ease of tracking them
> down (though CSS has long-standing arguments on how properties should
> be ordered, so to each their own on this).
>

Personally I'd rarely be tempted to put more than 9 or so arguments
directly into a function or class. Most of the time I can imagine unpacking
(or equiv.) would look much more readable in the circumstances that apply.

Back to comp.lang.python | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

In defence of 80-char lines Steven D'Aprano <steve+comp.lang.python@pearwood.info> - 2013-04-04 01:36 +0000
  Re: In defence of 80-char lines Andrew Berg <bahamutzero8825@gmail.com> - 2013-04-03 20:59 -0500
  Re: In defence of 80-char lines Mitya Sirenef <msirenef@lightbird.net> - 2013-04-03 22:40 -0400
  Re: In defence of 80-char lines llanitedave <llanitedave@veawb.coop> - 2013-04-03 21:32 -0700
    Re: In defence of 80-char lines Steven D'Aprano <steve+comp.lang.python@pearwood.info> - 2013-04-04 06:33 +0000
    Re: In defence of 80-char lines gregor <gregor@ediwo.com> - 2013-04-04 08:40 +0200
    Re: In defence of 80-char lines Peter Otten <__peter__@web.de> - 2013-04-04 08:43 +0200
    Re: In defence of 80-char lines Tim Chase <python.list@tim.thechases.com> - 2013-04-04 06:09 -0500
    Re: In defence of 80-char lines Roy Smith <roy@panix.com> - 2013-04-04 07:52 -0400
      Re: In defence of 80-char lines llanitedave <llanitedave@veawb.coop> - 2013-04-04 08:28 -0700
    Re: In defence of 80-char lines Jason Swails <jason.swails@gmail.com> - 2013-04-04 08:18 -0400
    Re: In defence of 80-char lines Joshua Landau <joshua.landau.ws@gmail.com> - 2013-04-04 18:18 +0100
    Re: In defence of 80-char lines Dennis Lee Bieber <wlfraed@ix.netcom.com> - 2013-04-04 18:22 -0400
  Re: In defence of 80-char lines rusi <rustompmody@gmail.com> - 2013-04-03 21:56 -0700
  Re: In defence of 80-char lines Rui Maciel <rui.maciel@gmail.com> - 2013-04-04 08:15 +0100
  Re: In defence of 80-char lines Jason Swails <jason.swails@gmail.com> - 2013-04-04 08:13 -0400
    Re: In defence of 80-char lines Roy Smith <roy@panix.com> - 2013-04-04 08:39 -0400
      Re: In defence of 80-char lines Jason Swails <jason.swails@gmail.com> - 2013-04-04 09:23 -0400
      Re: In defence of 80-char lines Neil Cerutti <neilc@norwich.edu> - 2013-04-04 15:56 +0000
      Re: In defence of 80-char lines Kushal Kumaran <kushal.kumaran+python@gmail.com> - 2013-04-04 23:04 +0530
        Re: In defence of 80-char lines Roy Smith <roy@panix.com> - 2013-04-04 19:55 -0400
  Re: In defence of 80-char lines Mitya Sirenef <msirenef@lightbird.net> - 2013-04-04 12:12 -0400
  Re: In defence of 80-char lines jmfauth <wxjmfauth@gmail.com> - 2013-04-04 13:28 -0700
    Re: In defence of 80-char lines Jason Swails <jason.swails@gmail.com> - 2013-04-04 17:00 -0400
    Re: In defence of 80-char lines Steven D'Aprano <steve+comp.lang.python@pearwood.info> - 2013-04-05 00:14 +0000

csiph-web