Hi Prasanta,
You have many lines of commented code in test case, which is not needed. Also there are cases of null pointer dereferencing at line 68 & 70 in test case. Please add checks for the same. And you can add bug evaluation in JBS for the same. Thanks, Jay From: Philip Race Sent: Wednesday, July 20, 2016 8:14 AM To: Prasanta Sadhukhan Cc: 2d-dev Subject: Re: [OpenJDK 2D-Dev] [9] RFR JDK-4882305: StreamPrintServ.getSupportedAttributeValues returns null for Orientation attr +1 -phil. On 7/14/16, 2:09 AM, Prasanta Sadhukhan wrote: Hi All, Please review a fix for an issue where it is seen that even though StreamPrintService returns OrientationRequested category as supported, when actually querying the supported attribute value with respect to the supported DocFlavors, null values are returned for the Orientation attributes when the DocFlavor is not either Pageable or Printable (SERVICE_FORMATTED) BUT we can print a jpeg iamge using StreamPrintService in LANDSCAPE mode and it worked fine so StreamPrintService should not return the supported values as null for supported DocFlavor. Bug: https://bugs.openjdk.java.net/browse/JDK-4882305 webrev: HYPERLINK "http://cr.openjdk.java.net/%7Epsadhukhan/4882305/webrev.00/"http://cr.openjdk.java.net/~psadhukhan/4882305/webrev.00/ Proposed fix is to add the image/jpeg, image/gif, image/png docflavor in addition to Pageable/Printable for Orientation attribute. Regards Prasanta
