Imporve custom compress speed via merging the multi Constraints and other optimizations as follows: #178
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue description:
bitmap.compress()
which is just like one(constraint) by one(constraint). For example, the code is as follows:if each constraint spend 2s, then the compression of the above will spend 10s(the sum of each time cost). The result is not what we want.
Solution:
ResolutionConstraints
andQualityConstraints
andFormatConstraints
(withoutDestinationConstraint
andSizeConstraint
) into only oneDefaultConstraint
.SizeConstraint
to the last post since it can reduce the number of compress times in this type ofSizeConstraint
.SizeConstraint
, we can avoid performing the compression if the bitmap compress format is png.DestinationConstraint
, we can reduce the number of copy file times via using the dest file fromDestinationConstraint
in the method ofcopyToCache
.