[PATCH] MarbleDebug: don't use a class extending QIODevice as a null device

Kai Koehne Kai.Koehne at qt.io
Thu Apr 20 03:19:35 PDT 2017


> -----Original Message-----
> From: Thiago Macieira [mailto:thiago at kde.org]
> [...]
> Sounds good, but do we need this setEnabled() in the first place? Is it used
> anywhere?

It's meant to be used by a filter installed by QLoggingCategory::installFilter (see the example http://doc.qt.io/qt-5/qloggingcategory.html#installFilter).

https://codereview.qt-project.org/#/c/192152/ aims to clarify this further in the documentation.

Regards

Kai



More information about the subsurface mailing list