Maybe. It's a curious case, since the optional quote would never match
(previous quantifier is greedy). I think you're probably still right though
- will add it
 On Sep 4, 2013 3:59 PM, "Ned Batchelder" <notifications@github.com> wrote:

> In diff_cover/diff_reporter.py:
>
> > @@ -147,7 +147,7 @@ def _git_diff(self):
> >          return self._diff_dict
> >
> >      # Regular expressions used to parse the diff output
> > -    SRC_FILE_RE = re.compile(r'^diff --git a/.* b/([^ \n]*)')
> > +    SRC_FILE_RE = re.compile(r'^diff --git "?a/.* "?b/([^ \n"]*)"?')
>
> This regex will work, but wouldn't it be clearer if it had an optional
> double-quote after the "a" file?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/diff-cover/pull/47/files#r6164802>
> .
>