On Tue, May 18, 2010 at 2:54 PM, Alexander Graf <ag...@suse.de> wrote: > Corentin Chary wrote: >> Create a new set_encoding() function to remove >> duplicated code in set_encodings(). >> >> Signed-off-by: Corentin Chary <corenti...@iksaif.net> >> > > Acked-by: Alexander Graf <ag...@suse.de> > > 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
