On Feb 16, 2016, at 6:42 PM, Joseph D. Darcy <joe.da...@oracle.com> wrote:
>> 1. Line 121: Would prefer “tiff” to “tif” as the former is the format >> acronym and consistent with the other array elements. > > I'm happy to change "tif" to "tiff" here; I verified the test passes under an > open-only build as a full Oracle build using "tiff”. Good - thanks. >> 2. Line 155: So a write failure appends to a log of failures on System.out >> as opposed to an actual test failure, or am I mistaken? >> > > Those are the combinations that need to be accommodated to allow open-only > Linux build to pass the tests. The relevant messages written out in this case > are: > > .... > Testing jpeg writer for type INT_RGB > Testing jpeg writer for type INT_BGR > Testing jpeg writer for type 3BYTE_BGR > Testing jpeg writer for type USHORT_565_RGB > Testing jpeg writer for type USHORT_555_RGB > Testing jpeg writer for type INT_ARGB > Can't write a type INT_ARGB BufferedImage! > Testing jpeg writer for type INT_ARGB_PRE > Can't write a type INT_ARGB_PRE BufferedImage! > Testing jpeg writer for type 4BYTE_ABGR > Can't write a type 4BYTE_ABGR BufferedImage! > Testing jpeg writer for type 4BYTE_ABGR_PRE > Can't write a type 4BYTE_ABGR_PRE BufferedImage! Alpha is not generally supported for JPEG so that looks OK. Thanks, Brian
