Path: csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail From: Chris Angelico Newsgroups: comp.lang.python Subject: Re: pylint woes Date: Sun, 8 May 2016 14:55:54 +1000 Lines: 20 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-Trace: news.uni-berlin.de 1KG9M7CTZgYe+1w/K/d57gGGe6KIu/4F0zQo4XAaS+tQ== Return-Path: X-Original-To: python-list@python.org Delivered-To: python-list@mail.python.org X-Spam-Status: OK 0.011 X-Spam-Evidence: '*H*': 0.98; '*S*': 0.00; 'builtin': 0.07; 'key.': 0.07; 'cc:addr:python-list': 0.09; '2016': 0.16; '2:40': 0.16; 'dfs': 0.16; 'from:addr:rosuav': 0.16; 'from:name:chris angelico': 0.16; 'received:io': 0.16; 'received:psf.io': 0.16; 'wrote:': 0.16; '>>>': 0.20; 'cc:2**0': 0.20; 'cc:addr:python.org': 0.20; 'leave': 0.23; 'header:In-Reply-To:1': 0.24; 'message- id:@mail.gmail.com': 0.27; 'function': 0.28; "skip:' 10": 0.28; 'says': 0.32; 'problem': 0.33; 'usually': 0.33; 'though.': 0.33; "i'll": 0.33; 'list': 0.34; 'received:google.com': 0.35; "isn't": 0.35; 'received:209.85': 0.36; 'pm,': 0.36; 'subject:: ': 0.37; 'received:209': 0.38; 'wrong': 0.38; 'called': 0.40; 'skip:u 10': 0.61; 'different': 0.63; 'chrisa': 0.84; 'to:none': 0.91 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:cc; bh=mI7dpa8NFuFzGhV2q1Ttvlj9Z7hHQdZsfmbcPMSPNWs=; b=AibvfmfpxHOHY+KD/jjHWaWY9VSOMgd0i/a5P0Ubvy2e5l1cfXhGgywWInYqWJWQOC gUbDCsVDBvqiLe5xdcRbNrxBFoxlENsYMXXQ714j+sOHUq2uonrBciOMRF4pxc/xijl4 ccmySiO8XsAlx/5LJEm1AxsbSVW98dK5I+anVRAVkBoo4YueyI1F9YkryCDz6CIF7yKX +5GCuM/8aOn016NCINc9x4RnSemGDT7J13TnM4qD/+LHspzeKxOMKKyKs8SGL3U2vA3C usThlUtXs6iPPi2sGzWsYUPRbIdXorzEN9bWu6LohxeZEC/18MQUM0hZmwBbG7+VdL5I k7ig== 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:cc; bh=mI7dpa8NFuFzGhV2q1Ttvlj9Z7hHQdZsfmbcPMSPNWs=; b=I/FidoE+kxFMbgrisAh3ZVdFuc0xQm1pqNgb36JmJ0y1zhWF7KBzcwVp5OcMRWF8c5 MpAOxRZTcaicotE5vXkrNfSZD7rezgmDt+/s3ADD7uMgErWGDJa/S3atn25sQjHZxMxz jouHJBRIlPqOJwGF3NgY0BaYVQf9EDwiRILEy9+jOWlkmmPN81DNSphaKC8lLrDyoC7h +vWULIJZSBS5MoC1KTTM9AWAZoC4BJKmJ50IO7n713s3wZXJ7S4H6rDG7CfE2frFb2Y8 sbeU2gQvqbiTHY1xsG6KV8bQWCRetzALJWO7ajvRE+9QWJLttmLckjhg9MVaJGitRaq2 PvLA== X-Gm-Message-State: AOPr4FUzRBfHZNtNy83HKgtXSA4r8HlY9ImoO5pAwtZtcfBVqfOOzIL/Qzl77n7mDIg9btH8APPlSPK7ML4Tlw== X-Received: by 10.25.79.12 with SMTP id d12mr12772989lfb.68.1462683355063; Sat, 07 May 2016 21:55:55 -0700 (PDT) In-Reply-To: X-BeenThere: python-list@python.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: General discussion list for the Python programming language List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Mailman-Original-Message-ID: X-Mailman-Original-References: Xref: csiph.com comp.lang.python:108338 On Sun, May 8, 2016 at 2:40 PM, DFS wrote: >>> It says "Used builtin function 'filter'. Using a list comprehension can >>> be >>> clearer. (bad-builtin)" >> >> >> Kill that message and keep using filter. > > > > Unfortunately, 'bad-builtin' caught 2 truly bad uses of built-ins (zip() and > id()), so I'll leave that warning in. > Hrm, that would be called "shadowing" built-ins, not bad use of them. Shadowing isn't usually a problem - unless you actually need id(), there's nothing wrong with using the name id for a database key. Very different from this message though. ChrisA