Comment 1:

Labels changed: added go1.1, removed go1.0.3.

Comment 2:

Labels changed: added size-m.

Comment 3:

https://golang.org/cl/7133051

Owner changed to @adg.

Status changed to Started.

Comment 4:

I just noticed that we already have CompilePOSIX. We might as well
have CompileLongest.
But again we need to fix the ab? problem too.

Comment 5:

I'm out of my depth as far as fixing the regexp engine.
Does this look correct to you?
  http://play.golang.org/p/rYOj6jVlbi

Comment 6 by Zarevucky.Jiri:

They are not equivalent. See http://code.google.com/p/re2/wiki/Syntax.
Alternative prefers shorter, ? always prefers longer variant.

Comment 7 by Zarevucky.Jiri:

According to http://code.google.com/p/re2/wiki/Syntax, they are not equivalent.
a? always prefers a to be present, but | prefers the first variant. So the equivalent
expression to ab? would be a(b|), and not a(|b).

Comment 8:

http://play.golang.org/p/rYOj6jVlbi is correct (ab ab ab a).
It's fine to just leave this open. I'll fix the regexp match (if there is a
bug, but it sure sounded like it on the review) once bigger stuff is done.
It will be easy.
Russ

Comment 9:

Labels changed: added priority-later, removed priority-soon.

Comment 10:

This issue was closed by revision f41ffc2.

Status changed to Fixed.