On 2010-08-20, Bruno Desthuilliers <bruno.42.desthuilli...@websiteburo.invalid> wrote: > make this : > > self.expiration_date = translate_date( > find(response, 'MPNExpirationDate').text, > '%Y-%m-%d', > '%m%d%Y' > ) > > I just HATE closing parens on the same line when the args don't > fit on one single line.
It's been interesting to see my code scrutinized like this. Thanks all. As a result of the discussion, I changed translate_date to trans_date_from_to to make the order of arguments clear. find is just a small wrapper around Element.find calls, inserting the namespace. -- Neil Cerutti -- http://mail.python.org/mailman/listinfo/python-list
