On Sat, Jul 20, 2013 at 11:37 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2013-07-19 at 15:13 -0700, Luis R. Rodriguez wrote: > >> -src_line = re.compile(r'^source\s+"?(?P<src>[^\s"]*)"?\s*$') >> +src_line = re.compile(r'^[\s]*source\s+"?(?P<src>[^\s"]*)"?\s*$') > > Looks fine, not that I understand why you added [] when you have just a > single character in there? Might be better to just write "\s*" (without > the quotes, obviously) Will update, thanks for the review. Lis -- 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
