8223813: (aio) Iocp.getErrorMessage should drop trailing \r\n

Roger Riggs Roger.Riggs at oracle.com
Tue Jun 11 19:58:51 UTC 2019


Hi Brian,

I'm fine with the current webrev too.  it does the job and is specific.

I agree about it not likely to be worth the effort.

Thanks, Roger



On 06/11/2019 03:56 PM, Brian Burkhalter wrote:
> Hi Roger,
>
> Thanks for the $0.02.
>
>> On Jun 11, 2019, at 12:14 PM, Roger Riggs <Roger.Riggs at Oracle.com 
>> <mailto:Roger.Riggs at Oracle.com>> wrote:
>>
>> Having the extraneous suffix consistently removed seems like a good 
>> thing.
>> Though I'm not sure what the best form of the utility function is:
>>  1) return the count of characters to remove
>>  2) just truncate the buffer.
>
> I am not sure either nor am I convinced that it would even be worth 
> the effort. As mentioned there are already getLastErrorString() and 
> getErrorString() for chars so I suppose the equivalent for WCHARs 
> would need to be added for at least one of these.
>
>> There may be ways to force some of the errors that would lead to the 
>> truncation, but testing all uses will not be practical.
>
> That’s what I thought.
>
> Thanks,
>
> Brian



More information about the nio-dev mailing list