Remove the capture name copies from the coalesce and simplify logic.
The capture name isn't needed for compilation and checking that it was
copied doesn't mean that the capture number was copied. Unfortunately,
neither ToString() nor Dump() surfaces the capture number, so I've set
"tripwires" to catch kRegexpCapture cap() == 0 in debug binary tests.
Reviewed-by: Paul Wankadia <email@example.com>
4 files changed