Merge pull request #36 from SDWebImage/fix_max_file_size_passes

dreampiggy · web-flow · commit e25194e655a7 · 2020-04-07T19:46:38.000+08:00
Fix that the max file size encoding need to increase the passes used by libwebp, we choose to use 6 from cwebp command line
diff --git a/SDWebImageWebPCoder/Classes/SDImageWebPCoder.m b/SDWebImageWebPCoder/Classes/SDImageWebPCoder.m
@@ -781,6 +781,7 @@ - (nullable NSData *)sd_encodedWebpDataWithImage:(nullable CGImageRef)imageRef q
     }
 
     config.target_size = (int)fileSize; // Max filesize for output, 0 means use quality instead
+    config.pass = fileSize > 0 ? 6 : 1; // Use 6 passes for file size limited encoding, which is the default value of `cwebp` command line
     config.thread_level = 1; // Thread encoding for fast
     config.lossless = 0; // Disable lossless encoding (If we need, can add new Encoding Options in future version)
     picture.use_argb = config.lossless; // Lossy encoding use YUV for internel bitstream

Original file line number	Diff line number	Diff line change
`@@ -781,6 +781,7 @@ - (nullable NSData *)sd_encodedWebpDataWithImage:(nullable CGImageRef)imageRef q`
`781`	`781`	`}`
`782`	`782`
`783`	`783`	`config.target_size = (int)fileSize; // Max filesize for output, 0 means use quality instead`
	`784`	+ config.pass = fileSize > 0 ? 6 : 1; // Use 6 passes for file size limited encoding, which is the default value of `cwebp` command line
`784`	`785`	`config.thread_level = 1; // Thread encoding for fast`
`785`	`786`	`config.lossless = 0; // Disable lossless encoding (If we need, can add new Encoding Options in future version)`
`786`	`787`	`picture.use_argb = config.lossless; // Lossy encoding use YUV for internel bitstream`