PATCH: swapped usage of idx_t and bm_word_t types in bitMap.inline.hpp
David Holmes
david.holmes at oracle.com
Tue Sep 17 05:48:46 PDT 2013
Hi Dan,
On 17/09/2013 10:17 PM, Dan Horák wrote:
> On Tue, 17 Sep 2013 12:39:22 +0200
> Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>
>> Hi Bengt,
>>
>> On Tue, 2013-09-17 at 11:48 +0200, Bengt Rutisson wrote:
>>> Hello Dan and Thomas,
>>>
>>> The change looks good to me to. Thanks for finding and fixing this!
>>>
>>
>> Thanks for the quick review.
>>
>> @Dan: The patch is now in the push queue and will go through the
>> hotspot-gc mercurial tree.
>
> thank you guys, any action still needed from me?
>
> There are other places in the hotspot code where int and size_t types
> are mixed (eg. in MAX2()/MIN2()) causing compile failures, so I'll
> submit another patch for these issues.
Have you actually eradicated all compile-time errors due to this issue?
I worry that things will just unwravel as you try to do this.
David
>
> Dan
>
More information about the hotspot-dev
mailing list