-
-
Notifications
You must be signed in to change notification settings - Fork 852
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
Improve CQ7 #678
Improve CQ7 #678
Conversation
Codecov Report
@@ Coverage Diff @@
## master #678 +/- ##
==========================================
- Coverage 85.18% 85.17% -0.02%
==========================================
Files 878 878
Lines 39122 39090 -32
Branches 2539 2539
==========================================
- Hits 33328 33293 -35
- Misses 5101 5104 +3
Partials 693 693
Continue to review full report at Codecov.
|
Ready for review. |
@@ -550,7 +550,8 @@ private void WriteApplicationHeader(ImageMetaData meta) | |||
private void WriteDefineHuffmanTables(int componentCount) | |||
{ | |||
// Table identifiers. | |||
byte[] headers = { 0x00, 0x10, 0x01, 0x11 }; | |||
Span<byte> headers = stackalloc byte[] { 0x00, 0x10, 0x01, 0x11 }; |
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.
We should keep an eye out for more things like this but I suspect you already have it covered 👍
Improve CQ7
Prerequisites
Description