Path: csiph.com!x330-a1.tempe.blueboxinc.net!newsfeed.hal-mli.net!feeder3.hal-mli.net!zen.net.uk!dedekind.zen.co.uk!aioe.org!feeder.news-service.com!news2.euro.net!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
Return-Path: <jjposner@codicesoftware.com>
X-Original-To: python-list@python.org
Delivered-To: python-list@mail.python.org
X-Spam-Status: OK 0.001
X-Spam-Evidence: '*H*': 1.00; '*S*': 0.00; 'subject:Python': 0.06; 'be:': 0.09; 'func': 0.09; 'header:In-reply-to:1': 0.09; 'none:': 0.09; 'wrote:': 0.15; '"if"': 0.16; "'\\n')": 0.16; '-john': 0.16; '50)': 0.16; 'received:167.206.4': 0.16; 'received:cv.net': 0.16; 'received:hcvlny.cv.net': 0.16; 'received:srv.hcvlny.cv.net': 0.16; 'cc:addr:python-list': 0.16; 'pm,': 0.16; 'def': 0.16; 'cc:addr:python.org': 0.30; 'cc:addr:gmail.com': 0.30; 'header :User-Agent:1': 0.34; 'received:192': 0.38; 'subject:: ': 0.38; 'cc:2**2': 0.38; 'perhaps': 0.39; 'should': 0.39; 'subject:with': 0.39; 'believe': 0.66
Date: Fri, 01 Jul 2011 11:18:29 -0400
From: John Posner <jjposner@codicesoftware.com>
Subject: Re: Using decorators with argument in Python
In-reply-to: <4E0B7D6A.1080404@stoneleaf.us>
To: Ethan Furman <ethan@stoneleaf.us>
MIME-version: 1.0
Content-type: text/plain; charset=ISO-8859-1
Content-transfer-encoding: 7BIT
References: <d35e1e82-410b-471b-a3c7-379c1822adb0@m22g2000yqh.googlegroups.com> <4e0a0a8b$1@dnews.tpgi.com.au> <4E0B5F7C.7030802@codicesoftware.com> <4E0B7D6A.1080404@stoneleaf.us>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.18) Gecko/20110616 Thunderbird/3.1.11
Cc: python-list@python.org, Lie Ryan <lie.1296@gmail.com>, poisonousrattle5@gmail.com
X-BeenThere: python-list@python.org
X-Mailman-Version: 2.1.12
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.542.1309535310.1164.python-list@python.org>
Lines: 31
NNTP-Posting-Host: 2001:888:2000:d::a6
X-Trace: 1309535310 news.xs4all.nl 21801 [2001:888:2000:d::a6]:38649
X-Complaints-To: abuse@xs4all.nl
Xref: x330-a1.tempe.blueboxinc.net comp.lang.python:8635

On 2:59 PM, Ethan Furman wrote:

<snip>

>     def __call__(self, func=None):
>         if func is None:
>             return self._call()
>         self.func = func
>         return self
>     def _call(self):
>         print("\n" + self.char * 50)
>         self.func()
>         print(self.char * 50 + '\n')
>

I believe the "if" block should be:

  if func is None:
      self._call()
      return

Or perhaps the _call() method should be revised:

  def _call(self):
      print("\n" + self.char * 50)
      retval = self.func()
      print(self.char * 50 + '\n')
      return retval

-John