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 21:53:05 UTC 2018


Hi Roger,

Thank you.  I am OK with the current version of the patch.

Happy Holidays!

Best
Lance
> On Dec 20, 2018, at 10:50 AM, Roger Riggs <Roger.Riggs at oracle.com> wrote:
> 
> Hi Lance,
> 
> Webrev:
>   http://cr.openjdk.java.net/~rriggs/webrev-8066619-5.patch/
> 
> Phillip does not yet have access to cr.openjdk.java.net.
> 
> $.02, Roger
> 
> On 12/20/2018 09:17 AM, Lance Andersen wrote:
>> 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>
>> 
>> 
>> 
> 

 <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