Review Request 113173: Reduce warnings noise when including wtf/Platform.h in KJS
Commit Hook
null at kde.org
Thu Oct 10 17:54:48 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113173/#review41529
-----------------------------------------------------------
This review has been submitted with commit 27dab0e58d5c93a911dd323374e7309ae4a9dec6 by Aleix Pol to branch frameworks.
- Commit Hook
On Oct. 8, 2013, 1:20 p.m., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113173/
> -----------------------------------------------------------
>
> (Updated Oct. 8, 2013, 1:20 p.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Repository: kdelibs
>
>
> Description
> -------
>
> Modifies how features are checked. This file has some pre-processor macros that are used to checking for OS features. These are defined with "1" if acceptable and undefined if the feature is not present. These macros are being checked with #if, so the pre-processor will complain whenever it's asked about one of the variables that haven't been defined.
>
> This patch adds a defined() first, before checking the value, so the pre-processor doesn't need to consider the variable value, if it's not present.
>
> This change makes it possible to read the compilation logs from modules that use KJS.
>
>
> Diffs
> -----
>
> tier1/kjs/src/wtf/Platform.h 843cfd2
>
> Diff: http://git.reviewboard.kde.org/r/113173/diff/
>
>
> Testing
> -------
>
> Builds, KJS tests pass.
>
>
> Thanks,
>
> Aleix Pol Gonzalez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131010/771ed183/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list