RFR: 8039368 Remove testcase from npt utf.c

[Dmitry Samersoff]

Staffan,

On 2014-04-09 16:20, Staffan Larsen wrote:
> I would prefer to keep the test where it is but change the “#if 1” to “#if 0” on line 399.

Are you OK with

 #ifdef COMPILE_WITH_UTF_TEST

-Dmitry

> 
> Thanks,
> /Staffan
> 
> On 9 apr 2014, at 14:03, Dmitry Samersoff <dmitry.samersoff at oracle.com> wrote:
> 
>> Staffan,
>>
>> We can move it to separate file and put to test directory (with no
>> intention to run it automatically).
>>
>> Other options is just change define to keep the test code but don't
>> compile it to production libnpt.
>>
>> -Dmitry
>>
>>
>> On 2014-04-09 11:12, Staffan Larsen wrote:
>>> It’s been a very useful little test when changing the code in utf.c. Not sure why it has to be removed.
>>>
>>> /Staffan
>>>
>>> On 8 apr 2014, at 17:30, Vladimir Kempik <vladimir.kempik at oracle.com> wrote:
>>>
>>>> Hello
>>>>
>>>> Please review this changeset:
>>>>
>>>> File jdk/src/share/npt/utf.c contains a testcase that shouldn't be in production library.
>>>>
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8039368
>>>> webrev: http://cr.openjdk.java.net/~vkempik/8039368/webrev.00/
>>>>
>>>> Thanks, Vladimir.
>>>>
>>>>
>>>
>>
>>
>> -- 
>> Dmitry Samersoff
>> Oracle Java development team, Saint Petersburg, Russia
>> * I would love to change the world, but they won't give me the sources.
> 


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.