On Mon, 2013-05-06 at 20:03 -0700, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" <mcgrof@xxxxxxxxxxxxxxxx> > > Add a top_header for cases where a patchset might have a > full header that we may want users to keep around. > # list of Patch objects > self.items = [] > + self.top_header = "" > > self.errors = 0 # fatal parsing errors > self.warnings = 0 # non-critical warnings > @@ -303,6 +304,7 @@ class PatchSet(object): > if headscan: > while not fe.is_empty and not fe.line.startswith("--- "): > header.append(fe.line) > + self.top_header += fe.line > fe.next() Looks fine to me, though I wonder if we could somehow get at that "header" list? Anatoly? johannes -- To unsubscribe from this list: send the line "unsubscribe backports" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html