<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="https://git.reviewboard.kde.org/r/115236/">https://git.reviewboard.kde.org/r/115236/</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;">This review has been submitted with commit 143d64206e65b57c0a245a65b3169ec25938431c by Alex Richardson to branch master.</pre>
<br />
<p>- Commit Hook</p>
<br />
<p>On January 22nd, 2014, 9:12 p.m. UTC, Alexander Richardson wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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.</div>
<div>By Alexander Richardson.</div>
<p style="color: grey;"><i>Updated Jan. 22, 2014, 9:12 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kio
</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;">3 Commits to get closer to compiling on windows:
1. Include qplatformdefs.h where possible
unistd.h and others are not available on e.g. Windows,
qplatformdefs.h includes the equivalent for each platform
2. add include(CheckLibraryExists) to CMakeLists.txt
On Linux this is apparently pulled in by some other file
whereas it is missing on Windows
3. Make KIO::MetaData completely inline to prevent linker errors on MSVC</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>src/ioslaves/ftp/ftp.h <span style="color: grey">(ad94979829a5d0e71ec57177b51f67e115c5445e)</span></li>
<li>src/ioslaves/ftp/ftp.cpp <span style="color: grey">(9ea642587bd754f0b4295fcb7d4db1b427f4326e)</span></li>
<li>src/ioslaves/help/kio_help.h <span style="color: grey">(0eab4ce1b393b48e552ca94d877f4c069450cee0)</span></li>
<li>src/ioslaves/http/http.cpp <span style="color: grey">(4f97b335c0e206f0a2ff8cfd515d0fae79614ad2)</span></li>
<li>src/ioslaves/http/http_cache_cleaner.cpp <span style="color: grey">(ffa8ab9580acf554b27027617cdac06b3f7755bf)</span></li>
<li>src/widgets/kdirmodel.cpp <span style="color: grey">(ce6e4c9aaddada5715b5aeef36ad163c77d3c635)</span></li>
<li>src/widgets/kpropertiesdialog.cpp <span style="color: grey">(8ddd37f0326e37109ce239a46bfa67d2f4c35411)</span></li>
<li>src/widgets/krun.cpp <span style="color: grey">(92dcfd8da04b9f3d80b632a37758017c088093ab)</span></li>
<li>src/widgets/kurlcompletion.cpp <span style="color: grey">(ed77fd3213db0524b5a934c94eb7645d98bd27c7)</span></li>
<li>tests/kioslavetest.cpp <span style="color: grey">(dc56240dc166dfa93ec099db5cdd262cc06249d4)</span></li>
<li>tests/kruntest.cpp <span style="color: grey">(68720ddf788e6b44468baf18dd04937094741274)</span></li>
<li>CMakeLists.txt <span style="color: grey">(9894ad5e1696fe31d8a3f065d29747cde0474d2c)</span></li>
<li>autotests/fileundomanagertest.cpp <span style="color: grey">(4ddee49eed254be6379a1302af2dd17aae28c15c)</span></li>
<li>autotests/kurlcompletiontest.cpp <span style="color: grey">(10048e2e15059596e446b03cedb0b302bf038cd5)</span></li>
<li>src/core/chmodjob.cpp <span style="color: grey">(b60cb9bf3d3b1a71a795ee5db7c72253dbdf0ad2)</span></li>
<li>src/core/kacl.h <span style="color: grey">(3c9c88d0408c855d02eaacb34d14f45c55343eb2)</span></li>
<li>src/core/kacl.cpp <span style="color: grey">(201b30a02ef93a137b9a8507cabaece6926d8739)</span></li>
<li>src/core/kfileitem.h <span style="color: grey">(553dbf8ccf96983803d0224cf8dc9d72f0107b6a)</span></li>
<li>src/core/kfileitem.cpp <span style="color: grey">(fdc0fc0279713887dc18ce1da8d3b00d422f6a9b)</span></li>
<li>src/core/kprotocolmanager.cpp <span style="color: grey">(a8746a4f9a78e6a01eaed6a70e99146ff40129d2)</span></li>
<li>src/core/krecentdocument.cpp <span style="color: grey">(ad0a97e7c1af85a94bb7483124b5035f0fca38a6)</span></li>
<li>src/core/metadata.h <span style="color: grey">(cd62e3b009c21485537ddd5449a6c347748c3caf)</span></li>
<li>src/core/metadata.cpp <span style="color: grey">(ad05032dff58314a305256993fcc7fe7b94751bc)</span></li>
<li>src/core/slave.h <span style="color: grey">(43b5cd8e8aa11da55d6f8a8d9fdf4eb3d0780d11)</span></li>
<li>src/core/slave.cpp <span style="color: grey">(ef9b3c7a57b4f4384343c4d4296ec88add857eb6)</span></li>
<li>src/core/slavebase.cpp <span style="color: grey">(a7ac4d5a6a87c256da9a3947b7223649927eba39)</span></li>
<li>src/core/slaveinterface.h <span style="color: grey">(d75eb6b02374375a73e1aaa57d5c2979a3bc365f)</span></li>
<li>src/core/slaveinterface.cpp <span style="color: grey">(faa4bd7dc1b5a733f0060f60626e8a0313bfea8a)</span></li>
<li>src/core/slaveinterface_p.h <span style="color: grey">(0ed4980a06f8aa8b3fc8c0717e4a88991483e98a)</span></li>
<li>src/filewidgets/kfileplaceeditdialog.cpp <span style="color: grey">(a7c433c8baecca8082f8d80077deab68e4b0cec2)</span></li>
<li>src/filewidgets/knewfilemenu.cpp <span style="color: grey">(dfc086b6e98b965739c062d7291bbd9139633beb)</span></li>
<li>src/ioslaves/file/file.h <span style="color: grey">(453298159791a78d877c4d81d2be73db073db3f2)</span></li>
<li>src/ioslaves/file/file.cpp <span style="color: grey">(e3ede0daa8054fc33d12acb9966a707f9484cd98)</span></li>
<li>src/ioslaves/file/file_win.cpp <span style="color: grey">(53e0f8f133bd5c4cbbd07afa945854efffc7297c)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/115236/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>