RFR: JDK-8211343: nsk_jvmti_parseoptions should handle multiple suboptions

gary.adams at oracle.com gary.adams at oracle.com
Mon Dec 31 18:06:02 UTC 2018


Here's a revised webrev.

   Webrev: 
http://bussund0416.us.oracle.com/export/users/gradams/work/webrevs/8211343/webrev.01/

Updates in this round of changes :
   - replaced index() with strchr() to avoid platform dependent issues 
with strings.h include
   - removed NSK_JVMTI_OPTION_VAL_SEP
   - removed temporary debugging print statements
   - removed empty options string from SetNativeMethodPrefix001
   - added free for temporary strdup buffer
   - updated copyright for 2019

On 12/21/18 1:52 PM, Gary Adams wrote:
> Here is a first pass at a replacement parser for jvmti test options.
>
>   Webrev: http://cr.openjdk.java.net/~gadams/8211343/webrev.00/
>   Issue: https://bugs.openjdk.java.net/browse/JDK-8211343
>
> Testing is in progress. Need to check out more platforms.
> On local linux testing one jvmti test failed SetNativeMethodPrefix001
> which passed an empty options string.
> ...




More information about the serviceability-dev mailing list