Path: csiph.com!v102.xanadu-bbs.net!xanadu-bbs.net!feeder.erje.net!eu.feeder.erje.net!weretis.net!feeder1.news.weretis.net!news.roellig-ltd.de!open-news-network.org!cyclone03.ams2.highwinds-media.com!news.highwinds-media.com!voer-me.highwinds-media.com!post01.fr7!fx10.am4.POSTED!not-for-mail From: alister Subject: Re: PEP 8 : Maximum line Length : Newsgroups: comp.lang.python User-Agent: Pan/0.139 (Sexual Chocolate; GIT bf56508 git://git.gnome.org/pan2) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Lines: 64 Message-ID: NNTP-Posting-Host: 82.5.246.13 X-Complaints-To: http://netreport.virginmedia.com X-Trace: 1399974255 82.5.246.13 (Tue, 13 May 2014 09:44:15 UTC) NNTP-Posting-Date: Tue, 13 May 2014 09:44:15 UTC Organization: virginmedia.com Date: Tue, 13 May 2014 09:44:15 GMT X-Received-Body-CRC: 1687688992 X-Received-Bytes: 2329 Xref: csiph.com comp.lang.python:71459 On Tue, 13 May 2014 10:45:49 +0200, Peter Otten wrote: > Ganesh Pal wrote: > >> Hi Team , >> >> >> what would be the best way to intent the below line . >> >> I have few lines in my program exceeding the allowed maximum line >> Length of 79./80 characters >> >> Example 1 : >> >> p = >> > Subprocess.Popen(shlex.split (cmd),stdout=subprocess.PIPE,stderr=subprocess.PIPE) >> >> >> Iam running pylint and it says the above line is tool long how do I >> limit it to 79 character without violating any rules >> >> ************* Module isi_corrupt C: 14,0: Line too long (88/80) >> W: 19,0: Bad indentation. Found 6 spaces, expected 8 > > (1) Newlines are allowed inside an open (, [, or {. So: > > p = subprocess.Popen( > shlex.split(cmd), stdout=subprocess.PIPE, > stderr=subprocess.PIPE) > > Other techniques: > > (2) Introduce helper variables: > > cmd = shlex.split(cmd) > p = subprocess.Popen(cmd, stdout=subprocess.PIPE, > stderr=subprocess.PIPE) > > (3) Import names: > > from subprocess import PIPE p = subprocess.Popen(shlex.split(cmd), > stdout=PIPE, stderr=PIPE) > > (4) Use aliases: > > import subprocess as sp p = sp.Popen(shlex.split(cmd), stdout=sp.PIPE, > stderr=sp.PIPE) or simply leave it alone Pep 8 is a guide not a "Set in Stone" rule & even pep 8 does say that the 80 character limit can be exceeded if breaking the line would make the code less readable -- Catastrophic failure of the IDE cable???. What are you doing to the poor thing, jumping on it? - Beau Kuiper on linux-kernel