On Thu, Aug 25, 2011 at 13:17, Gabriel Charette <gch...@google.com> wrote:

> What do you mean? the || is aligned with the 'marker' entry above it.
> Do you want the || to be aligned? i.e.
> marker == PPH_RECORD_IREF
> || marker == PPH_RECORD_XREF
> || marker == PPH_RECORD_PREF
> ?

Yeah, this way.  Every operand of the predicate in its own line.

> Ya I had the same thought and same resolution. Relying on include_ix == -1U
> was already sketchy imo, and now adding another trigger on stream == NULL
> was just too much assumptions and dependence on the rest of the
> implementation I thought.

Sounds good.


Diego.

Reply via email to