Request for review 8003722: More gcc 4.7 compilation errors

David Holmes david.holmes at oracle.com
Tue Nov 20 18:52:40 PST 2012


Sorry but I must protest. After the last fix we were returned to an 
unpleasant mixture of using directives and this-> augmentation. The fix 
that was lost had applied the using directive.

Can we please use the using directive and just fix these once and for all.

Otherwise get rid of all using directives. But lets have a consistent 
approach here.

Thanks,
David

On 21/11/2012 11:44 AM, Coleen Phillimore wrote:
> Summary: Add a few more this->qualifications.
> Reviewed-by: coleenp
> Contributed-by: duboscq at ssw.jku.at
>
> http://cr.openjdk.java.net/~coleenp/8003722/
>
> Ran runThese tests with CMS gc. I want to check this into hotspot-gc
> since it has the other changes (hotspot-rt doesn't yet).
>
> Thanks,
> Coleen


More information about the hotspot-dev mailing list