<div dir="ltr"><div class="gmail_default" style="font-family:monospace,monospace;font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On 10 October 2016 at 23:32, Albert Astals Cid <span dir="ltr"><<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">El dissabte, 8 d’octubre de 2016, a les 7:49:21 CEST, Ben Cooksley va escriure:<br>
</span><span class="">> Hi all,<br>
><br>
> Just a heads up that at the request of Jaroslaw Staniek, Sysadmin has<br>
> moved the following repositories to KDE Review:<br>
><br>
> - KDb (kdb)<br>
<br>
</span>kdb seems to have some public/exported classes without d-pointer that may make keeping BC harder, is that on purpose?<br></blockquote><div><br><div style="font-family:monospace,monospace;font-size:small" class="gmail_default">​Yes, many of them can be converted. There are some performance-critical points like the ones around drivers and connection. <br><br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
You may also want to run clazy over it, there's some interesting things in the report,  i.e.<br>
<br>
/home/kdeunstable/kdb/src/<wbr>parser/generated/KDbToken.h:<wbr>88:28: warning: Missing reference on non-trivial type (class KDbToken) [-Wclazy-function-args-by-ref]<br>
    inline bool operator==(KDbToken other) const { return v == other.v; }<br>
                           ^<br>
/home/kdeunstable/kdb/src/<wbr>parser/generated/KDbToken.h:<wbr>91:28: warning: Missing reference on non-trivial type (class KDbToken) [-Wclazy-function-args-by-ref]<br>
    inline bool operator!=(KDbToken other) const { return v != other.v; }<br>
<br>
<br>
The full log is at<br>
  <a href="https://paste.kde.org/pojcgtvh2" rel="noreferrer" target="_blank">https://paste.kde.org/<wbr>pojcgtvh2</a><br>
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<br>
<br></blockquote><div><br><div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">Thanks. Serious warnings will be fixed.<br>Fixing other stuff won't hurt, ​</div> <div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">​yes, some stuff like this is a good task for students, will have to find them eventually. The stuff passed Coverity checks not ​so long ago. So far only gcc and msvc were the compilers in daily use, clang maybe year ago.<br></div><br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Cheers,<br>
  Albert<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">regards, Jaroslaw Staniek<br><br>KDE:<br>: A world-wide network of software engineers, artists, writers, translators<br>: and facilitators committed to Free Software development - <a href="http://kde.org" target="_blank">http://kde.org</a><br>Calligra Suite:<br>: A graphic art and office suite - <a href="http://calligra.org" target="_blank">http://calligra.org</a><br>Kexi:<br>: A visual database apps builder - <a href="http://calligra.org/kexi" target="_blank">http://calligra.org/kexi</a><br>Qt Certified Specialist:<br>: <a href="http://www.linkedin.com/in/jstaniek" target="_blank">http://www.linkedin.com/in/jstaniek</a></div>
</div></div>