[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-runtime-dev
Subject:    Re: RFR(XS) 8230168 Use ClasspathStream for FileMapInfo::create_path_array
From:       Ioi Lam <ioi.lam () oracle ! com>
Date:       2019-08-26 19:42:01
Message-ID: c154a8d2-cfe0-e710-78f2-e082f8596050 () oracle ! com
[Download RAW message or body]

Thanks Lois & Fred for the review!

- Ioi

On 8/26/19 12:05 PM, Lois Foltan wrote:
> +1.
> Lois
>
> On 8/26/2019 2:24 PM, Frederic Parain wrote:
>> Looks good to me.
>>
>> Fred
>>
>>
>>> On Aug 26, 2019, at 12:49, Ioi Lam <ioi.lam@oracle.com> wrote:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8230168
>>> http://cr.openjdk.java.net/~iklam/jdk14/8230168-classpath-stream-for-filemap-cpp.v01/ 
>>>
>>>
>>>
>>> Just a follow-up to JDK-8230011 [1]
>>> There's similarly duplicated in filemap.cpp that can be replaced by 
>>> ClasspathStream
>>>
>>> Thanks
>>> - Ioi
>>>
>>>
>>> [1] https://bugs.openjdk.java.net/browse/JDK-8230011
>>>
>>>
>

[prev in list] [next in list] [prev in thread] [next in thread]