<div dir="ltr">2008/8/28 David Faure <span dir="ltr"><<a href="mailto:faure@kde.org">faure@kde.org</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Sunday 24 August 2008, Thiago Macieira wrote:<br>
> >in mandriva we use lzma to compress some tarballs, manpages, ... so we<br>
> >needed to add support to lzma into KDE, which have been done by Per<br>
> >Øyvind Karlsen<br>
</div><div class="Ih2E3d">> ><a href="http://kenobi.mandriva.com/%7Eneoclust/kdelibs-add-lzma-support.patch" target="_blank">http://kenobi.mandriva.com/~neoclust/kdelibs-add-lzma-support.patch</a><br>
<br>
</div>Very cool. When I wrote FilterBase as base class for gzip + bzip2, I was wondering<br>
"what are the odds that there is ever going to be another compression library<br>
with a similar API?"  -- turns out that lzma has rather similar API so it fits --- very nice!<br>
I like the implementation of readHeader too :-)<br>
<br>
Patch looks good (apart from the things Thiago pointed out already)<br>
<font color="#888888"><br>
</font></blockquote></div>It needs to be put on hold though, this was submitted a bit prematurely upstream as the API of new lzma utils has changed since,<br>might change a bit more and possibly other changes as well. On the positive side, these changes killed off C99 dependence in headers.<br>
<br>I'll create and submit a new patch once the API has stabilized for real and Mandriva Linux 2009.0 is done. :)<br><br>--<br>Regards,<br>Per Øyvind Karlsen<br></div>