[libcxx-commits] [PATCH] D62451: Regex backreference [1/3] Fixes backreferences for extended grammar.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante marked 2 inline comments as done.
Mordante added inline comments.


================
Comment at: libcxx/include/regex:3544
+                            __push_back_ref(__val);
+                            __first = ++__temp;
+                    }
----------------
zoecarver wrote:
> Is the intent here that `__first = ++__temp` will be executed regardless of the value of `__val`, or that it will only be executed if `__val` is greater than `1` and less than `9`? If the former, I think you should unindent the statement, as it makes the intent confusing. If the latter, then you need braces around the if statement. 
Good catch, still not entirely used to not attaching braces to all ifs. So forgot them when needed. Will upload a new patch later today.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62451/new/

https://reviews.llvm.org/D62451