* Frederic Weisbecker <fweis...@gmail.com> wrote: > > Btw., a side note, append_chain() is a rather confusing function in > > itself, with logic-inversion gems like: > > > > if (!found) > > found = true; > > The check is pointless yeah, I'll remove that.
Are you sure it can be removed as-is? It inverts the 'found' condition, which then can be used later on AFAICS. (I haven't looked very deeply though.) Thanks, Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/