On Tue, Apr 30, 2019 at 09:48:14PM +0300, Alexander Korotkov wrote:
> I'd like to add couple of comments from my side.

> > -       returns an error because the second template string space is 
> > consumed
> > -       by the letter <literal>J</literal> in the input string.
> > +       returns an error because the second space in the template string 
> > consumes
> > +       the letter <literal>M</literal> from the input string.
> 
> Why <literal>M</literal>?  There is no letter "M" is input string.
> The issue here is that we already consumed "J" from "JUN" and trying
> to match "UN" to "MON".  So, I think we should live
> <literal>J</literal> here.  The rest of this change looks good.

Seems like I confused myself while resolving rebase conflict.

Thanks for checking.

Justin


Reply via email to