<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 />
<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>
<br />
<p>- Kevin</p>
<br />
<p>On May 1st, 2013, 9:37 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 1, 2013, 9:37 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;">Change uses of QCoreApplication::translate() with no context to QObject::tr() in tier1 frameworks.
There are no changes needed in tier 2.
Issues outstanding:
* What do we do with the QT_TRANSLATE_NOOP_3 macro usage that we put in when changing it to QCoreApplication::translate() previously? I'd propose leaving it as is, because it seems like it would still work the same, but maybe that's not the neatest plan
* kconfig/src/core/kconfig.cpp uses QCoreApplication::translate with the "KConfig" context in one instance. Should this be left alone?</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/solid/src/solid/backends/kupnp/internetgatewaydevice1.cpp <span style="color: grey">(2cff30781764dcadde045fbed1f8a5ea24aa1b1f)</span></li>
<li>tier1/solid/src/solid/backends/hal/haldevice.cpp <span style="color: grey">(8e433d9a96a9e15245f0c2d735950dfca8730ae6)</span></li>
<li>tier1/solid/src/solid/backends/fstab/fstabmanager.cpp <span style="color: grey">(46415e0776fcef4a8576ccc1f57417f5465d5484)</span></li>
<li>tier1/kcodecs/src/kencodingprober.cpp <span style="color: grey">(3008f3ba5fc833ae999194f7d33b9a8d95170398)</span></li>
<li>tier1/kcodecs/src/kcharsets.cpp <span style="color: grey">(45f80b609abfdd131eded0cbe91e2287973046ca)</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>