<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jan 21, 2014 at 12:26 AM, David Gil Oliva <span dir="ltr"><<a href="mailto:davidgiloliva@gmail.com" target="_blank">davidgiloliva@gmail.com</a>></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"><div dir="ltr"><div><div><div><div><div>Hi!<br><br></div>
<div>
<u><b>kcompletionuitest.cpp</b></u><br></div><div><br></div>In kcompletionuitest.cpp I have found many uses of QStringLiteral that perhaps aren't necessary, but I'm not sure:<br>
<br>Form1::Form1(QWidget *parent)<br>    : QWidget(parent)<br>{<br>    setAttribute(Qt::WA_DeleteOnClose);<br>    <b>setObjectName(QStringLiteral("Form1"));</b><br>    resize(559, 465);<br>    <b>setWindowTitle(QStringLiteral("Form1"));</b><br>


    Form1Layout = new QVBoxLayout(this);<br><br>    GroupBox1 = new QGroupBox(this);<br>    GroupBox1->setLayout(new QVBoxLayout());<br>    <b>GroupBox1->setTitle(QStringLiteral("Completion Test"));</b><br>


<br></div>Can't it be just GroupBox1->setTitle("Completion Test"); for example?<br><br></div><div><u><b>kcomboboxtest.cpp</b></u><br></div><div><br></div>Another thing: the test kcomboboxtest has a Enable/Disable button that uses a QTimer, so, when pressed, it takes 5 endless seconds to enable/disable the combo boxes and change the text of the button. May I take off that QTimer or is it useful in some unknown way? :-/<br>


<br><br></div>Thanks!<span class=""><font color="#888888"><br><br></font></span></div><span class=""><font color="#888888">David Gil<br></font></span></div>
<br>_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
<br></blockquote></div><br></div><div class="gmail_extra">You'll see you'll get errors if you pass the char* right away to the QString. This is because we don't let frameworks use the QString(char*) constructor as it's not the fastest way to construct a QString. For applications we don't enforce it, though.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">For more information, see the Qt documentation or <a href="http://woboq.com/blog/qstringliteral.html">http://woboq.com/blog/qstringliteral.html</a></div><div class="gmail_extra">

<br></div><div class="gmail_extra">Regarding the QTimer, I'd assume that it's the thing that does the actual testing. Maybe you want to reduce the timeout?</div><div class="gmail_extra"><br></div><div class="gmail_extra">

Aleix</div></div>