Review Request: Use unordered_map when gxx >= 4.3 to fix a bunch of warnings.
Milian Wolff
mail at milianw.de
Thu May 6 22:44:29 UTC 2010
> On 2010-05-04 21:54:10, Andreas Pakulat wrote:
> > Hmm, I'd rather drop older compiler support than adding a bunch of ifdef's and cmake-checks, but gcc 4.2 is "only" 2 years old, which means all LTS distributions will still have it (or even gcc 4.1). So fine with me for now (but we should re-evaluate this in a year).
closing as committed
- Milian
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3889/#review5419
-----------------------------------------------------------
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