[Development] Orphan modules

[Gatis Paeglis]

> APIs have modern replacements in Qt.


Not everything can be or should be in cross platform code.


> other ways to do the same?


For some maybe, but it is all about convenience. Some things (e.g. peeking at our internal event queue) can be done only if we expose that info, there isn't really other ways.


> I could see kwin_x11 needing it, but I really don't see all the other applications doing so.


There actually are other application that need this. Xlib used to provide various higher level APIs on top of X11 protocol. XCB, which stands for X11 C Binding, is low level API, which does not provide equivalents for all of Xlib APIs that some Qt 4 applications, that use native calls, depend on. This is where QX11Extras is used to provide some alternatives.


My only concern was that it should not be a separate module. The topic here was about orphan modules, everything else (e.g. if any of APIs have become redundant) should be discussed elsewhere.


> I was going to say we needed replacement API for it, but I realise the
QX11EmbedContainer is not there. Since people have lived for the last 6 years
without it in Qt 5, it doesn't seem we really need a replacement.


See https://codereview.qt-project.org/#/c/42990/


Gatis Paeglis.


________________________________
From: Development <development-bounces+gatis.paeglis=qt.io at qt-project.org> on behalf of Jean-Michaël Celerier <jeanmichael.celerier at gmail.com>
Sent: Thursday, September 13, 2018 8:37:57 AM
To: Thiago Macieira
Cc: development
Subject: Re: [Development] Orphan modules

There are quite a bunch of people using it out there : https://github.com/search?l=C%2B%2B&q=QX11Info+NOT+tst_QX11Info+NOT+QT_END_LICENSE&type=Code


-------
Jean-Michaël Celerier
http://www.jcelerier.name


On Thu, Sep 13, 2018 at 1:41 AM Thiago Macieira <thiago.macieira at intel.com<mailto:thiago.macieira at intel.com>> wrote:
On Wednesday, 12 September 2018 10:09:53 PDT Tor Arne Vestbø wrote:
> If they do, does the list say anything about whether or not those APIs have
> modern replacements in Qt or other ways to do the same?

There's exactly one class in QtX11Extras: QX11Info.
http://doc.qt.io/qt-5/qx11info.html

I could see kwin_x11 needing it, but I really don't see all the other
applications doing so. Do we have a replacement for QX11Info?

--
Thiago Macieira - thiago.macieira (AT) intel.com<http://intel.com>
  Software Architect - Intel Open Source Technology Center



_______________________________________________
Development mailing list
Development at qt-project.org<mailto:Development at qt-project.org>
http://lists.qt-project.org/mailman/listinfo/development
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.qt-project.org/pipermail/development/attachments/20180913/0fb8bde6/attachment.html>