RFR: 8054823: Add size_t as a valid VM flag type

[Stefan Karlsson]

On 11/08/14 23:09, Vladimir Ivanov wrote:
> test/gc/arguments/TestArrayAllocatorMallocLimit.java
> 25  * @test TestArrayAlocatorMallocLimit at test
>
> Typo.

Fixed.

>
> Otherwise, looks good.

Great!

thanks,
StefanK
>
> Best regards,
> Vladimir Ivanov
>
> On 8/11/14, 10:46 AM, Stefan Karlsson wrote:
>> On 2014-08-11 18:35, Vladimir Kozlov wrote:
>>> Stefan,
>>>
>>> Seems fine to me,
>>
>> Thanks for reviewing!
>>
>>> except new tests don't have correct @bug.
>>>
>>> 27  * The test helsp verifying that size_t flags can be set/read.
>>>                ^ typo
>>>
>>> SizeTTest.java - remove @author
>>
>> Too much copy-n-paste. I've updated the patch and did some additional
>> cleanups to the jtreg annotations:
>> http://cr.openjdk.java.net/~stefank/8054823/webrev.01.delta/
>> http://cr.openjdk.java.net/~stefank/8054823/webrev.01/
>>
>> thanks,
>> StefanK
>>
>>>
>>> Thanks,
>>> Vladimir
>>>
>>> On 8/11/14 6:48 AM, Stefan Karlsson wrote:
>>>> Hi all,
>>>>
>>>> I propose that we add 'size_t' as a valid VM flag type. This will
>>>> help with type safety, since we won't have to cast
>>>> back and forth between the two types.
>>>>
>>>> This patch adds the framework, but only changes one of the flags. I
>>>> choose to change one of our experimental flags, so
>>>> that I could write tests for this change.
>>>>
>>>> Please, review this patch:
>>>>   http://cr.openjdk.java.net/~stefank/8054823/webrev.00/
>>>>   https://bugs.openjdk.java.net/browse/JDK-8054823
>>>>
>>>> thanks,
>>>> StefanK
>>