On Tue, May 18, 2010 at 2:54 PM, Alexander Graf <[email protected]> wrote: > Corentin Chary wrote: >> Create a new set_encoding() function to remove >> duplicated code in set_encodings(). >> >> Signed-off-by: Corentin Chary <[email protected]> >> > > Acked-by: Alexander Graf <[email protected]> > > Keep in mind that this still means that the last requested compression > algorithm is used, which is reverse according to spec. >
I didn't notice that the loop was reversed, and that the right encoding was selected *before* my patch. I think the right thing to do is to reverse my previous patch and add a comment at the begining of the loop. I can also just make a patch to reverse the loop. -- Corentin Chary http://xf.iksaif.net
