Hi Frédéric, On Fri, Mar 29, 2013, Frédéric Dalleau wrote: > --- a/sbc/sbc_primitives.h > +++ b/sbc/sbc_primitives.h > @@ -40,6 +40,7 @@ struct sbc_encoder_state { > int position; > /* Number of consecutive blocks handled by the encoder */ > uint8_t increment; > + bool odd; Since most of the code related on this variable uses function pointers would it make sense to simply play with those pointers and have the odd function set the pointer to the even function and vice versa? If this is a performance critical part of the code it'd remove one if-statement. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html