ryan/mozjpeg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

TJBench: Don't override subsamp until args parsed

Browse Source 
Otherwise, passing -subsamp after -lossless might cause the worst-case
JPEG buffer size to be too small.
This commit is contained in:
DRC
2024-08-28 18:00:14 -04:00
parent c72bbd9c06
commit d62079717c
2 changed files with 10 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
java/TJBench.java
View File

@@ -936,10 +936,9 @@ final class TJBench {
System.out.println("Using arithmetic entropy coding\n");
arithmetic = true;
xformOpt |= TJTransform.OPT_ARITHMETIC;
} else if (argv[i].equalsIgnoreCase("-lossless")) {
} else if (argv[i].equalsIgnoreCase("-lossless"))
lossless = true;
subsamp = TJ.SAMP_444;
} else if (argv[i].equalsIgnoreCase("-rgb"))
else if (argv[i].equalsIgnoreCase("-rgb"))
pf = TJ.PF_RGB;
else if (argv[i].equalsIgnoreCase("-rgbx"))
pf = TJ.PF_RGBX;
@@ -1130,6 +1129,9 @@ final class TJBench {
precision != 12)
System.out.println("Computing optimal Huffman tables\n");
if (lossless)
subsamp = TJ.SAMP_444;
if (precision == 16 && !lossless)
throw new Exception("-lossless must be specified along with -precision 16");
if (precision != 8 && doYUV)

8
tjbench.c
View File

@@ -1057,10 +1057,9 @@ int main(int argc, char *argv[])
printf("Using arithmetic entropy coding\n\n");
arithmetic = 1;
xformOpt |= TJXOPT_ARITHMETIC;
} else if (!strcasecmp(argv[i], "-lossless")) {
} else if (!strcasecmp(argv[i], "-lossless"))
lossless = 1;
subsamp = TJSAMP_444;
} else if (!strcasecmp(argv[i], "-rgb"))
else if (!strcasecmp(argv[i], "-rgb"))
pf = TJPF_RGB;
else if (!strcasecmp(argv[i], "-rgbx"))
pf = TJPF_RGBX;
@@ -1200,6 +1199,9 @@ int main(int argc, char *argv[])
if (optimize && !progressive && !arithmetic && !lossless && precision != 12)
printf("Computing optimal Huffman tables\n\n");
if (lossless)
subsamp = TJSAMP_444;
if (precision == 16 && !lossless) {
printf("ERROR: -lossless must be specified along with -precision 16\n");
retval = -1; goto bailout;