On Fri, Jul 27, 2018 at 11:05 PM Stefan Beller <sbel...@google.com> wrote: > emit_line_0 has no nested conditions, but puts out all its arguments > (if set) in order. There is still a slight confusion with set > and set_sign, but let's defer that to a later patch. > > 'first' used be output always no matter if it was 0, but that got lost > got lost at e8c285c4f9c (diff: add an internal option to dual-color > diffs of diffs, 2018-07-21), as there we broadened the meaning of 'first' > to also signal an early return.
s/used be/used to be/ redundant "got lost" > The change in 'emit_line' makes sure that 'first' is never content, but > always under our control, a sign or special character in the beginning > of the line (or 0, in which case we ignore it). > > Signed-off-by: Stefan Beller <sbel...@google.com>
