Moved to KDE Review: KDb, KReport, KProperty
Albert Astals Cid
aacid at kde.org
Mon Oct 10 22:32:20 BST 2016
El dissabte, 8 d’octubre de 2016, a les 7:49:21 CEST, Ben Cooksley va escriure:
> Hi all,
>
> Just a heads up that at the request of Jaroslaw Staniek, Sysadmin has
> moved the following repositories to KDE Review:
>
> - KDb (kdb)
kdb seems to have some public/exported classes without d-pointer that may make keeping BC harder, is that on purpose?
You may also want to run clazy over it, there's some interesting things in the report, i.e.
/home/kdeunstable/kdb/src/parser/generated/KDbToken.h:88:28: warning: Missing reference on non-trivial type (class KDbToken) [-Wclazy-function-args-by-ref]
inline bool operator==(KDbToken other) const { return v == other.v; }
^
/home/kdeunstable/kdb/src/parser/generated/KDbToken.h:91:28: warning: Missing reference on non-trivial type (class KDbToken) [-Wclazy-function-args-by-ref]
inline bool operator!=(KDbToken other) const { return v != other.v; }
The full log is at
https://paste.kde.org/pojcgtvh2
in case you're interested, note i'm not asking to "fix" everything, some stuff is a bit "too much" but having a look at it won't hurt
Cheers,
Albert
More information about the kde-core-devel
mailing list