Path: csiph.com!usenet.pasdenom.info!news.albasani.net!newsfeed.freenet.ag!news2.euro.net!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail Return-Path: X-Original-To: python-list@python.org Delivered-To: python-list@mail.python.org X-Spam-Status: OK 0.023 X-Spam-Evidence: '*H*': 0.95; '*S*': 0.00; '"""': 0.05; 'python': 0.09; 'oh,': 0.09; '2.7:': 0.16; '4:59': 0.16; 'caching': 0.16; 'subject:search': 0.16; 'wrote:': 0.17; 'basically': 0.17; 'exists': 0.17; 'module,': 0.17; 'thu,': 0.17; '>>>': 0.18; 'feb': 0.19; 'import': 0.21; 'setup.': 0.22; 'header:In-Reply-To:1': 0.25; 'looks': 0.26; 'message-id:@mail.gmail.com': 0.27; 'subject:/': 0.28; 'all.': 0.28; "d'aprano": 0.29; 'steven': 0.29; 'function': 0.30; 'skip:8 10': 0.32; 'to:addr:python-list': 0.33; 'received:google.com': 0.34; 'done': 0.34; 'whatever': 0.35; 'pm,': 0.35; 'received:209.85': 0.35; 'does': 0.37; 'being': 0.37; 'received:209': 0.37; 'subject:: ': 0.38; 'to:addr:python.org': 0.39; 'url:a': 0.72; 'savings': 0.75; 'absolutely': 0.84; '2013': 0.84; 'subject:via': 0.84; 'thing...': 0.84; 'to:name:python': 0.84 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:content-type; bh=sNOuy9B3UcNBkm/hNUZg/c2vomGakCee5xVs9btLiEs=; b=b1lfsWnzp7/RXEzuEI5Wdl61k0NFszUyYnBjQAJkXniBZajc4mwBympLMMvOFkoWzL ozzO3NrwLdfVRYWzmH3zFcXd+uBZmpzusONWzEwFjuNZUPKAFQrz0aRosMmHpx7hW8pu 3yMOtgExNrLsGvNKpdFmyspVYz/GF3irojKphHYsjdnoTRWI5ifzJEekzxFdlk7jIw1h 5OxnxsHevTK5OoG9QL79GmkUc4SAwUYhjUBmXGRYLMB8x5lvN7J3PuDGx9GvIjlgVrbR VSx5IFxzBE78soG5JNPCrqXHBqgRiJA5svtSVgBUM0j2SKrYVz6s9xKCfeTV/1BOuWUo MZvg== X-Received: by 10.66.87.67 with SMTP id v3mr11204938paz.63.1360284972366; Thu, 07 Feb 2013 16:56:12 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <51143feb$0$29974$c3e8da3$5496439d@news.astraweb.com> References: <511319c7$0$21812$c3e8da3$76491128@news.astraweb.com> <51142e96$0$6512$c3e8da3$5496439d@news.astraweb.com> <51143feb$0$29974$c3e8da3$5496439d@news.astraweb.com> From: Ian Kelly Date: Thu, 7 Feb 2013 17:55:31 -0700 Subject: Re: Curious to see alternate approach on a search/replace via regex To: Python Content-Type: text/plain; charset=ISO-8859-1 X-BeenThere: python-list@python.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: General discussion list for the Python programming language List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Newsgroups: comp.lang.python Message-ID: Lines: 29 NNTP-Posting-Host: 2001:888:2000:d::a6 X-Trace: 1360284981 news.xs4all.nl 6855 [2001:888:2000:d::a6]:36187 X-Complaints-To: abuse@xs4all.nl Xref: csiph.com comp.lang.python:38389 On Thu, Feb 7, 2013 at 4:59 PM, Steven D'Aprano wrote: > Oh, one last thing... pulling out "re.compile" outside of the function does > absolutely nothing. You don't even compile anything. It basically looks up > that a compile function exists in the re module, and that's all. Using Python 2.7: >>> t1 = Timer(""" ... nx = re.compile(r'https?://(.+)$') ... v = nx.search(u).group(1) ... ux = re.compile(r'([-:./?&=]+)') ... ux.sub('_', v)""", """ ... import re ... u = 'http://alongnameofasite1234567.com/q?sports=run&a=1&b=1'""") >>> t2 = Timer(""" ... v = nx.search(u).group(1) ... ux.sub('_', v)""", """ ... import re ... nx = re.compile(r'https?://(.+)$') ... ux = re.compile(r'([-:./?&=]+)') ... u = 'http://alongnameofasite1234567.com/q?sports=run&a=1&b=1'""") >>> min(t1.repeat()) 11.625409933385388 >>> min(t2.repeat()) 8.825254885746652 Whatever caching is being done by re.compile, that's still a 24% savings by moving the compile calls into the setup.