<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 22, 2015 at 9:44 PM, Daniel Vrátil <span dir="ltr"><<a href="mailto:dvratil@kde.org" target="_blank">dvratil@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi all,<br>
<br>
I'd like to ask for review of another Framework from kdepimlibs: KF5Syndication<br>
<br>
KF5Syndication is an RSS/Atom parsing library. It also provides API to fetch<br>
feeds directly from network.<br>
<br>
It's a Tier 3 Framework (depends on KCodecs and KIO). AFAIK it's currently being<br>
used only by Akregator.<br>
<br>
I would like to submit KF5Syndication for the standard 2 week review<br>
period, and if everything is OK, then move it to Frameworks.<br>
<br>
Cheers,<br>
Daniel<br><span><font color="#888888"><br></font></span></blockquote><div>Disclaimer: I've never seen this code nor knew about it's existence till ~30 minutes ago.</div><div><br></div><div>This library depends on QtXml (quite heavily in fact). That very Qt module is deprecated [1].</div><div>Funnily enough, that is the only place that mentions it as being deprecated. No QtXml classes mention it in the docs and you won't get any compiler warnings either when you decide to use it.</div><div><br></div><div>When searching for bug reports for deprecating QtXml i found [2] where they say it's been mislabeled and should have the same status as Widgets. Aka: done.</div><div>Which also means it might go deprecated in some future release (Qt6?).</div><div><br></div><div>Now i don't know what the real state of QtXml is, but if it really is deprecated then i don't know if it's wise to release this library while depending on a deprecated module.</div><div>In that case it might be better to "port" (which probably requires quite some work) to the QXmlStreamReader and QXmlStreamWriter.</div><div><br></div><div>Just my 5 cents :)</div><div><br></div><div>[1] <a href="http://doc.qt.io/qt-5/qtmodules.html" target="_blank">http://doc.qt.io/qt-5/qtmodules.html</a></div><div>[2] <a href="https://bugreports.qt.io/browse/QTBUG-32926" target="_blank">https://bugreports.qt.io/browse/QTBUG-32926</a></div><div> </div></div><br></div></div>