[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-frameworks-devel
Subject:    Re: New framework: KF5Syndication
From:       Mark Gaiser <markg85 () gmail ! com>
Date:       2015-04-22 20:51:15
Message-ID: CAPd6JnHsw=kitGKpJyW6bOppQ=ytMTnw8tM9SS-diH-yBhayjw () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Wed, Apr 22, 2015 at 9:44 PM, Daniel Vr=C3=A1til <dvratil@kde.org> wrote=
:

> Hi all,
>
> I'd like to ask for review of another Framework from kdepimlibs:
> KF5Syndication
>
> KF5Syndication is an RSS/Atom parsing library. It also provides API to
> fetch
> feeds directly from network.
>
> It's a Tier 3 Framework (depends on KCodecs and KIO). AFAIK it's currentl=
y
> being
> used only by Akregator.
>
> I would like to submit KF5Syndication for the standard 2 week review
> period, and if everything is OK, then move it to Frameworks.
>
> Cheers,
> Daniel
>
> Disclaimer: I've never seen this code nor knew about it's existence till
~30 minutes ago.

This library depends on QtXml (quite heavily in fact). That very Qt module
is deprecated [1].
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.

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.
Which also means it might go deprecated in some future release (Qt6?).

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.
In that case it might be better to "port" (which probably requires quite
some work) to the QXmlStreamReader and QXmlStreamWriter.

Just my 5 cents :)

[1] http://doc.qt.io/qt-5/qtmodules.html
[2] https://bugreports.qt.io/browse/QTBUG-32926

[Attachment #5 (text/html)]

<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">&lt;<a href="mailto:dvratil@kde.org" \
target="_blank">dvratil@kde.org</a>&gt;</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&#39;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&#39;s a Tier 3 Framework (depends on KCodecs and KIO). AFAIK it&#39;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&#39;ve never seen this code nor knew about it&#39;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&#39;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&#39;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&#39;t know what the real state of \
QtXml is, but if it really is deprecated then i don&#39;t know if it&#39;s wise to \
release this library while depending on a deprecated module.</div><div>In that case \
it might be better to &quot;port&quot; (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>


[Attachment #6 (text/plain)]

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic