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