RFR: JDK-8066619: String(byte[],int,int,int) in String has been deprecated in Manifest and Attributes
Lance Andersen
lance.andersen at oracle.com
Thu Dec 20 14:17:47 UTC 2018
Hi Philipp,
The tests look better. Any chance you can post the webrev in the future vs the patch as it makes it easer to review.
Best
Lance
> On Dec 19, 2018, at 1:24 AM, Philipp Kunz <philipp.kunz at paratix.ch> wrote:
>
> now with assertThrows. Find a new patch attached.
>
> On Tue, 2018-12-18 at 13:38 -0800, Martin Buchholz wrote:
>> On Tue, Dec 18, 2018 at 1:13 PM Lance Andersen <lance.andersen at oracle.com <mailto:lance.andersen at oracle.com>>
>> wrote:
>>
>>>
>>>
>>>
>>> Is there a reason you did not use
>>>
>>> @Test(expectedExceptions = IOException.class)
>>>
>>> It just seems to make the test a bit more readable
>>>
>>> That construct has lost popularity over the past decade, sort of like
>>
>> inheritance is no longer considered a silver bullet.
> <8066619.patch>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev
mailing list