-
-
Notifications
You must be signed in to change notification settings - Fork 851
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Webp improvements #1846
Webp improvements #1846
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1846 +/- ##
======================================
Coverage 87% 87%
======================================
Files 937 937
Lines 48423 48438 +15
Branches 6056 6057 +1
======================================
+ Hits 42225 42317 +92
+ Misses 5192 5114 -78
- Partials 1006 1007 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
public void Dispose() | ||
{ | ||
this.Dispose(disposing: true); | ||
GC.SuppressFinalize(this); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If your types can be sealed I would do so and avoid SuppressFinalize
where it is not necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I think the Dispose(bool disposing)
pattern is only favorable when we actually have a finalizer or expect subclasses to have a finalizer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job 👍
Prerequisites
Description
This PR changes the following:
CostInterval
objects by pooling a small amount of them. The number ofCostInterval
objects can easily get into 100k+ for an 1 megapixel image.Related to #1786
Allocations master:
Allocations PR:
Test was done with lossless encoding of Calliphora image.