<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/110264/">http://git.reviewboard.kde.org/r/110264/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 2nd, 2013, 6:43 a.m. UTC, <b>Kevin Ottens</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I'd indeed leave the QT_TRANSLATE_NOOP3 uses in even if the redefine won't make it for the beauty contest. :-)
I guess it's the price to pay to have that working with our infra.
As for the use of QCoreApplication::translate in kconfig, the context doesn't bring much here, it could be changed to tr() just fine IMO.
Rest of the patch looks fine to me, if it includes the change in kconfig it can go in IMO.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The use of QCoreApplication::translate in kconfig should stay in light of what Ossi pointed out in his review.</pre>
<br />
<p>- George</p>
<br />
<p>On May 3rd, 2013, 8:32 p.m. UTC, George Goldberg 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 and David Faure.</div>
<div>By George Goldberg.</div>
<p style="color: grey;"><i>Updated May 3, 2013, 8:32 p.m.</i></p>
<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;">Fix QObject::tr use in solid.
Fix translation in kcodecs. Uses Q_DECLARE_TR_FUNCTIONS.
Change uses of QCoreApplication::translate() with no context to QObject::tr() in tier1 frameworks.</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;">Everything still compiles.</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>tier1/kcodecs/src/kcharsets.h <span style="color: grey">(a075e5348b6ca9305cf77f3d05062fed2ca5b229)</span></li>
<li>tier1/kcodecs/src/kcharsets.cpp <span style="color: grey">(45f80b609abfdd131eded0cbe91e2287973046ca)</span></li>
<li>tier1/kcodecs/src/kencodingprober.h <span style="color: grey">(d6a940b275317f75e53f097e25bc92ecf2fd4435)</span></li>
<li>tier1/kcodecs/src/kencodingprober.cpp <span style="color: grey">(3008f3ba5fc833ae999194f7d33b9a8d95170398)</span></li>
<li>tier1/solid/src/solid/backends/fstab/fstabmanager.cpp <span style="color: grey">(46415e0776fcef4a8576ccc1f57417f5465d5484)</span></li>
<li>tier1/solid/src/solid/backends/hal/haldevice.cpp <span style="color: grey">(8e433d9a96a9e15245f0c2d735950dfca8730ae6)</span></li>
<li>tier1/solid/src/solid/backends/kupnp/internetgatewaydevice1.cpp <span style="color: grey">(2cff30781764dcadde045fbed1f8a5ea24aa1b1f)</span></li>
<li>tier1/solid/src/solid/backends/kupnp/kupnprootdevice.cpp <span style="color: grey">(664e72e5fbffb246402f7f1c87132125f7d87720)</span></li>
<li>tier1/solid/src/solid/backends/kupnp/mediaserver1.cpp <span style="color: grey">(28e8267afefe23382898dac5c86242db41746d6f)</span></li>
<li>tier1/solid/src/solid/backends/kupnp/mediaserver2.cpp <span style="color: grey">(9967680429accc7fe7e45a73d65e7c33a6b308d5)</span></li>
<li>tier1/solid/src/solid/backends/kupnp/mediaserver3.cpp <span style="color: grey">(3805a5c52d89043046d66c2aaaf01b3699e5a13f)</span></li>
<li>tier1/solid/src/solid/backends/udev/udevdevice.cpp <span style="color: grey">(e00c37c816cb1c7c94a3460341f5b0b13088eb3a)</span></li>
<li>tier1/solid/src/solid/backends/udev/udevmanager.cpp <span style="color: grey">(1446868cca7a42155478be0dc75be2fd47725e7f)</span></li>
<li>tier1/solid/src/solid/backends/udisks/udisksdevice.cpp <span style="color: grey">(e1c47afc952bf232b9fce1ffe053cce1b3b80a67)</span></li>
<li>tier1/solid/src/solid/backends/udisks/udisksmanager.cpp <span style="color: grey">(31e581656d98c0e103d6ceb605b3b0bfb4b00a54)</span></li>
<li>tier1/solid/src/solid/backends/udisks2/udisksdevice.cpp <span style="color: grey">(1aa7b6d139be3ba5dc6d78e1ae55b2d0841dbfbd)</span></li>
<li>tier1/solid/src/solid/backends/udisks2/udisksmanager.cpp <span style="color: grey">(e76dfd1c94b39009ae4522ce9d3f535fc2c5eac2)</span></li>
<li>tier1/solid/src/solid/backends/upnp/upnpdevicemanager.cpp <span style="color: grey">(e3cf29954da081bb5f23b859310f4f5a57e2702c)</span></li>
<li>tier1/solid/src/solid/backends/upower/upowerdevice.cpp <span style="color: grey">(c61bee940cf5e8295d99f055582317e7e626eedd)</span></li>
<li>tier1/solid/src/solid/backends/upower/upowermanager.cpp <span style="color: grey">(ee198a0ae0da44e35cc911e9fd830a78241bc867)</span></li>
<li>tier1/solid/src/solid/backends/wmi/wmibattery.cpp <span style="color: grey">(4b72f80514afa7fd7bbfe706d231ec669ed2f8d4)</span></li>
<li>tier1/solid/src/solid/backends/wmi/wmidevice.cpp <span style="color: grey">(0e85ae9c31eaab8c863e86845733dff17453456a)</span></li>
<li>tier1/solid/src/solid/deviceinterface.cpp <span style="color: grey">(0cfc8e84ddbcbf31b44e881055ae173a6c589fdb)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110264/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>