[OpenJDK 2D-Dev] [9]: RFR JDK-8058316, , lookupDefaultPrintService returns null on Solaris 11

2016-03-03 Thread prasanta sadhukhan
Hi Phil, Bug: https://bugs.openjdk.java.net/browse/JDK-8058316 webrev: http://cr.openjdk.java.net/~psadhukhan/8058316/webrev.00/ The issue was on some Solaris 11 hosts lookupDefaultPrintService() returns null while lpstat -d reports an ipp printer. The issue was found to exist in linux too sin

Re: [OpenJDK 2D-Dev] [9]: RFR JDK-8058316, , lookupDefaultPrintService returns null on Solaris 11

2016-03-03 Thread Yuri Nesterenko
Thank you, Prasanta! Could you please add this or similar explanation to that JDK-8058316? Please use all the words/phrases a desperate tester would search. If we cannot have a regression test we should have enough search terms to find this bug in case of (apparent) regression. -yan On 03/03/20

Re: [OpenJDK 2D-Dev] [9]: RFR JDK-8058316, , lookupDefaultPrintService returns null on Solaris 11

2016-03-03 Thread Phil Race
I think there is a small memory leak with this code : https://www.cups.org/documentation.php/doc-2.0/api-cups.html#cupsGetDests "Use the cupsFreeDests function to free the destination list" Also I am not sure if you are using cupsGetDest as intended :- 202 cups_dest_t *dest = j2d_cupsGetDe

Re: [OpenJDK 2D-Dev] [9]: RFR JDK-8058316, , lookupDefaultPrintService returns null on Solaris 11

2016-03-04 Thread prasanta sadhukhan
Hi Phil, Please find the modified webrev http://cr.openjdk.java.net/~psadhukhan/8058316/webrev.01/ Regards Prasanta On 3/4/2016 3:21 AM, Phil Race wrote: I think there is a small memory leak with this code : https://www.cups.org/documentation.php/doc-2.0/api-cups.html#cupsGetDests "Use the cup

Re: [OpenJDK 2D-Dev] [9]: RFR JDK-8058316, , lookupDefaultPrintService returns null on Solaris 11

2016-03-04 Thread Phil Race
Approved. -phil. On 03/04/2016 02:47 AM, prasanta sadhukhan wrote: Hi Phil, Please find the modified webrev http://cr.openjdk.java.net/~psadhukhan/8058316/webrev.01/ Regards Prasanta On 3/4/2016 3:21 AM, Phil Race wrote: I think there is a small memory leak with this code : https://www.cups.

Re: [OpenJDK 2D-Dev] [9]: RFR JDK-8058316, , lookupDefaultPrintService returns null on Solaris 11

2016-03-06 Thread Jayathirth D V
-8058316, , lookupDefaultPrintService returns null on Solaris 11 Approved. -phil. On 03/04/2016 02:47 AM, prasanta sadhukhan wrote: > Hi Phil, > > Please find the modified webrev > http://cr.openjdk.java.net/~psadhukhan/8058316/webrev.01/ > > Regards > Prasanta > On 3/4