Review Request: Use unordered_map when gxx >= 4.3 to fix a bunch of warnings.
Bertjan Broeksema
b.broeksema at home.nl
Tue May 4 20:21:38 UTC 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3889/#review5402
-----------------------------------------------------------
trunk/extragear/sdk/kdevelop/languages/cpp/CMakeLists.txt
<http://reviewboard.kde.org/r/3889/#comment5111>
Yes, trailing whitespace, already fixed locally.
- Bertjan
On 2010-05-04 20:20:53, Bertjan Broeksema wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3889/
> -----------------------------------------------------------
>
> (Updated 2010-05-04 20:20:53)
>
>
> Review request for KDevelop, Milian Wolff and David Nolden.
>
>
> Summary
> -------
>
> Fix a bunch of compile warnings for gnu c++ >= 4.3. I chose for the patch which is as less intrusive as possible. Ideally we just replace the whole hash_map with QHash. However, I don't have time to profile the difference. David, any reason why you didn't use QHash in in particular?
>
>
> Diffs
> -----
>
> trunk/extragear/sdk/kdevelop/languages/cpp/CMakeLists.txt 1122832
> trunk/extragear/sdk/kdevelop/languages/cpp/parser/parser.h 1122832
>
> Diff: http://reviewboard.kde.org/r/3889/diff
>
>
> Testing
> -------
>
> Compiled with both cases for GCC_IS_NEWER_THAN_4_3, works for me and with unordered_map I don't get the warnings anymore.
>
>
> Thanks,
>
> Bertjan
>
>
More information about the KDevelop-devel
mailing list