[9] RFR of 8065556: (bf) Buffer.position and other methods should include detail in IAE
Brian Burkhalter
brian.burkhalter at oracle.com
Thu Apr 23 17:12:31 UTC 2015
On Apr 23, 2015, at 9:38 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> The approach looks okay to me. It would have been interesting to have done micro benchmarks to see it matches Martin's observation/bug as that was news to me.
I could do that if you like. I need to brush up on JMH anyway as I used a lot during a certain period of time but that’s a while ago now.
> In Buffer then it would be nice to reduce the method names a bit so that the declarations don't spill over two lines. I realize it's just nit picking but it's good to keep the code consistent when you can.
Understood. I’ll see what I can do. I also did not like this sort of style:
private IllegalArgumentException createSomeException( // indent == 4
Object someParameter, …) // indent == 8
> The test update looks okay now.
Good.
Thanks,
Brian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20150423/1715c58c/attachment.html>
More information about the nio-dev
mailing list