<div dir="ltr"><div class="gmail_default" style="color:#000000"><br></div><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 4, 2013 at 7:09 AM, Tim Chase <span dir="ltr"><<a href="mailto:python.list@tim.thechases.com" target="_blank">python.list@tim.thechases.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2013-04-04 08:43, Peter Otten wrote:<br>
<div class="im">> llanitedave wrote:<br>
>> self.mainLabel.SetFont(wx.Font(12, wx.DEFAULT, wx.NORMAL, wx.BOLD, faceName = "FreeSans"))<br>
><br>
</div><div class="im">> I think I would prefer<br>
><br>
> labelfont = wx.Font(<br>
>     pointSize=12,<br>
>     style=wx.DEFAULT,<br>
>     family=wx.NORMAL,<br>
>     weight=wx.BOLD,<br>
>     faceName="FreeSans")<br>
> self.mainLabel.SetFont(labelfont)<br>
<br>
</div>+1<br>
The only change I'd make to this suggestion would be to add a<br>
semi-superfluous comma+newline after the last keyword argument too:<br>
<div class="im"><br>
 labelfont = wx.Font(<br>
     pointSize=12,<br>
     style=wx.DEFAULT,<br>
     family=wx.NORMAL,<br>
     weight=wx.BOLD,<br>
</div>     faceName="FreeSans",<br>
     )<br>
<br>
which makes diffs cleaner when you need to insert something after<br>
faceName:<br>
<br>
--- peter1.txt  2013-04-04 06:03:01.420762566 -0500<br>
+++ peter2.txt  2013-04-04 06:03:34.736762582 -0500<br>
@@ -3,4 +3,5 @@<br>
     style=wx.DEFAULT,<br>
     family=wx.NORMAL,<br>
     weight=wx.BOLD,<br>
-    faceName="FreeSans")<br>
+    faceName="FreeSans",<br>
+    otherValue=42)<br>
<br>
vs.<br>
<br>
--- tkc1.txt    2013-04-04 06:02:52.436762562 -0500<br>
+++ tkc2.txt    2013-04-04 06:03:51.392762588 -0500<br>
@@ -4,4 +4,5 @@<br>
     family=wx.NORMAL,<br>
     weight=wx.BOLD,<br>
     faceName="FreeSans",<br>
+    otherValue=42,<br>
     )<br></blockquote><div><br></div><div style="color:rgb(0,0,0)" class="gmail_default">+1</div><div style="color:rgb(0,0,0)" class="gmail_default"><br></div><div style="color:rgb(0,0,0)" class="gmail_default">I wasn't aware you could do this (superfluous trailing commas), although I admit it hadn't occurred to me to try.  I use git for virtually everything, and I regularly parse diffstats -- this would make them much easier to grok.  (It's an incredibly helpful bug-tracking technique)</div>
<div style="color:rgb(0,0,0)" class="gmail_default"><br></div><div style="color:rgb(0,0,0)" class="gmail_default">Thanks!</div><div style="color:rgb(0,0,0)" class="gmail_default">Jason</div></div></div></div>