<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/113302/">http://git.reviewboard.kde.org/r/113302/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 19th, 2013, 8:58 a.m. UTC, <b>David Faure</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/113302/diff/1/?file=202321#file202321line128" style="color: black; font-weight: bold; text-decoration: underline;">staging/kio/src/filewidgets/kencodingfiledialog.h</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">public:</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">125</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">static</span> <span class="n">Result</span> <span class="nf">getOpenFileNameAndEncoding</span><span class="p">(</span><span class="k">const</span> <span class="n">QString</span><span class="o"><span class="hl">&</span></span><span class="hl"> </span><span class="n">encoding</span><span class="o"><span class="hl">=</span></span><span class="n">QString</span><span class="p">(),</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">123</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">static</span><span class="hl"> </span><span class="n"><span class="hl">KIO_EXPORT</span></span> <span class="n">Result</span> <span class="nf">getOpenFileNameAndEncoding</span><span class="p">(</span><span class="k">const</span> <span class="n">QString</span> <span class="o"><span class="hl">&</span></span><span class="n">encoding</span> <span class="o"><span class="hl">=</span></span><span class="hl"> </span><span class="n">QString</span><span class="p">(),</span></pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">If you export the class again, no need to export the static methods individually.
Or was it intended to make the class private, even with this new solution? (in that case it should be documented, the ctor should be private, etc.)</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Yes, the intention was to only export the static methods. I'll make the ctor private and document a bit.</pre>
<br />
<p>- Teo</p>
<br />
<p>On October 17th, 2013, 2:04 p.m. UTC, Teo Mrnjavac wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Frameworks, David Faure and Kevin Ottens.</div>
<div>By Teo Mrnjavac.</div>
<p style="color: grey;"><i>Updated Oct. 17, 2013, 2:04 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdelibs
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This makes KEncodingFileDialog static, moves it to staging/kio and moves some stuff from KFileDialog to KFileWidget to reduce duplication.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Seems to work fine in a test app.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kio/CMakeLists.txt <span style="color: grey">(ac80e393c877280dd8033ec13e8e772afea6d2f9)</span></li>
<li>kio/kfile/kencodingfiledialog.h <span style="color: grey">(abb939abeb000126005f1a1a9c6fd50b8bd322bd)</span></li>
<li>kio/kfile/kencodingfiledialog.cpp <span style="color: grey">(d55d908473aae2859838d88fd776cc65cecf3317)</span></li>
<li>kio/kfile/kfiledialog.cpp <span style="color: grey">(32eb98a4490a31c5ed51150ca3cb7af5375dc67e)</span></li>
<li>staging/kio/src/filewidgets/CMakeLists.txt <span style="color: grey">(e8d8c2edf11ad6352e13eb6e7436f828a4a55e3a)</span></li>
<li>staging/kio/src/filewidgets/kencodingfiledialog.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>staging/kio/src/filewidgets/kfilewidget.h <span style="color: grey">(f7b162ab39b997274b21f9eff0c6374545e0a9ad)</span></li>
<li>staging/kio/src/filewidgets/kfilewidget.cpp <span style="color: grey">(824ac563f3f8c463426f0a45e792f107ac402a40)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/113302/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>