<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 76f474c43b80c5bc7c79403def8ee0443c3ea1e5 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>