On Apr 5, 6:01 pm, Neil Hodgson <[EMAIL PROTECTED]> wrote: > [EMAIL PROTECTED]: > > > else: > > my_match = capper_re.match(f): > > if my_match: > > capper = capper_re.match(f).group(1) > > if capper == 'JJ' or capper == 'JeffreyJacobs': > > capper = 'Jeffrey C. Jacobs' > > past_title = True > > The assignment to my_match here is not used, so the test can be "if > capper_re.match(f)" which can then merge up into the previous else as an > elif dropping one level of indentation. > > Neil
That was a typo. I meant to reuse my_match in the line "capper = my_match.group(1)" rather than the line above just so I would not have to evaluate the regular expression twice. Sorry for the confusion. Jeffrey. -- http://mail.python.org/mailman/listinfo/python-list