RFR 8156931: java.nio.Buffer tests cleanup
pavel.rappo at oracle.com
Fri May 13 14:19:33 UTC 2016
Thanks for looking into this! I have fixed excessively long lines and removed
clarifying curly braces where not needed.
> On 13 May 2016, at 13:00, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> On 13/05/2016 12:39, Pavel Rappo wrote:
>> Could you please review my change for JDK-8156931?
>> It's a pass through before updating tests for "(bf) Hoist slice and duplicate
>> methods up to java.nio.Buffer"
> This looks okay except that I wouldn't bother with the extra braces.
> Would you mind fixing testAlign while you are there? The tests for this introduce very long lines and so means annoying horizontal scrolling when looking at the diffs.
More information about the nio-dev