Merge C++ conversion into trunk (4/6 - hash table rewrite)

Michael Matz matz@suse.de
Wed Aug 15 14:05:00 GMT 2012


Hi,

On Wed, 15 Aug 2012, Richard Guenther wrote:

> Like the following, only the coverage.c use is converted.  I've never 
> seen template function arguments anywhere and having to repeat them all 
> over the place is really really ugly (yes, even if only in the 
> implementation).
> 
> This goes with static member functions and not wrapping any data. It 
> goes away with the requirement of having externally visible global 
> functions for the hash/compare functions as well (which was ugly 
> anyway).
> 
> Comments?

Well, it looks nicer than what's there currently.  As the element 
functions now are scoped and normal member functions, they should be named 
with lower case characters of course.  I do like that the hash table would 
then only have one real template argument; the Allocator, well, no better 
idea comes to my mind.


Ciao,
Michael.



More information about the Gcc-patches mailing list