[PATCH] fib-trie: code cleanup (v2)

From: Mathieu Desnoyers
Date: Tue Dec 08 2009 - 14:25:26 EST


* Stephen Hemminger (shemminger@xxxxxxxxxxxxxxxxxxxx) wrote:
> On Tue, 8 Dec 2009 13:48:39 -0500
> Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxx> wrote:
>
> > /* use cache location of last found key */
> > - if (iter->pos > 0 && pos >= iter->pos && (l = fib_find_node(t, iter->key)))
> > - pos -= iter->pos;
> > + if (iter->pos > 0 && pos >= iter->pos)
> > + l = fib_find_node(t, iter->key);
> > + if (l)
> > + pos -= iter->pos;
> > else {
>
> Missing bracket's here change the semantics.

Ouch, yes. thanks.


--
Mathieu Desnoyers
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F BA06 3F25 A8FE 3BAE 9A68
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/