RFR 8187631 : Refactor FileDescriptor close implementation
Roger Riggs
Roger.Riggs at Oracle.com
Tue Sep 19 18:01:03 UTC 2017
Hi,
I had a suggestion to use fdo and fd parameter names and to fix
copyright dates.
http://cr.openjdk.java.net/~rriggs/webrev-fd-refactor-8187631-02/
Thanks, Roger
On 9/19/2017 10:52 AM, Brian Burkhalter wrote:
> Hi Roger,
>
> On Sep 19, 2017, at 6:51 AM, Roger Riggs <Roger.Riggs at oracle.com
> <mailto:Roger.Riggs at oracle.com>> wrote:
>
>> The updated webrev is:
>> http://cr.openjdk.java.net/~rriggs/webrev-fd-refactor-8187631-02/
>>
>> I avoided cluttering the initial webrev with misc cleanup.
>
> This looks good. The name “rawfd” is good. It could equally well be
> named “nativefd” but that’s being picky.
>
>> I renamed the parameters, corrected some old <code></code> markup and
>> reordered
>> the declarations so a side-by side diff of the Unix and Windows
>> FileDescriptor.java files
>> makes it easy to see the necessary differences.
>
> All good.
>
More information about the core-libs-dev
mailing list