Review Request 126339: remove kdewin dependency
Patrick Spendrin
ps_ml at gmx.de
Mon Dec 14 10:37:27 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126339/
-----------------------------------------------------------
(Updated Dez. 14, 2015, 10:37 vorm.)
Review request for KDE Frameworks and Martin Tobias Holmedahl Sandsmark.
Changes
-------
there actually is a qsysinfo.h, not sure why I didn't find it in the first place.
Repository: khtml
Description
-------
This removes the direct kdewin dependency by replacing problematic
function calls (uname, snprintf) with their Qt counterparts.
Some leftover unix header includes removed too.
Diffs (updated)
-----
CMakeLists.txt 51fe02a01c8166046d1c6085ec4a5b6e617e1fea
src/ecma/kjs_binding.cpp 5e122f0f0d70e6734565e7ab205d14a92c79d287
src/ecma/kjs_navigator.cpp 2f7be8d11e5af84e08ac640ccbc97f70aeac8abd
src/ecma/kjs_proxy.h 24abd1e1bac0932f8829f02185953142c74aadc8
src/ecma/kjs_proxy.cpp 20430f48fce986ca654c49c5771ad839845f11ab
src/khtml_pagecache.cpp 8e1841f6b0e816dfd8faa76f2191b269c4716011
src/khtml_part.cpp adbcd800a526e9f8fd92a553e62ee64791872938
src/kmultipart/kmultipart.cpp 1ad3bbb9b6d6a022799d5ef85f426fc9f911d45b
Diff: https://git.reviewboard.kde.org/r/126339/diff/
Testing
-------
Windows, Linux compiles.
Thanks,
Patrick Spendrin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151214/a545a4b9/attachment.html>
More information about the Kde-frameworks-devel
mailing list