> + if not file[0]+file[-2:] == '<]>': file = None > + if file is None: source_lines = None > + else: I don't see a reason to put these on a single line. > + if not source_lines: > + source_lines = None Especially when this similar line is treated differently. Eric.