On Wed, 10 Mar 2021 22:49:13 -0500, Alex Karle wrote:

> >         This would now be clearer as:
> > 
> >             if (qlen > 0 && qp->q_text[qlen - 1] == '\\') {
>
> I agree this is clearer, but I think we'll need a `qlen = 0`
> initialization up above the while loop if we go with this variation
> (otherwise an uninitialized qlen could be > 0, causing indexing a null
> q_text, right?). Apologies if this was implied by your comment, thought
> it couldn't hurt to flag!

You are absolutely correct.

 - todd

Reply via email to