virtualenv/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-08-10 06:10:52 +02:00
Code Issues Releases Activity

avcodec/mjpegenc_common: Don't check luma/chroma matrices unnecessarily

Browse Source 
These matrices are only used for MJPEG, not for LJPEG.
So only check them for the former. This is in preparation
for removing said matrices from LJPEG altogether
(i.e. sending NULL matrices).

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
This commit is contained in:
Andreas Rheinhardt
2022-10-05 18:23:06 +02:00
parent 6bf99f8c93
commit 77adbe28ab
1 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
libavcodec/mjpegenc_common.c
View File

@@ -61,15 +61,13 @@ static void jpeg_table_header(AVCodecContext *avctx, PutBitContext *p,
ScanTable *intra_scantable, ScanTable *intra_scantable,
uint16_t luma_intra_matrix[64], uint16_t luma_intra_matrix[64],
uint16_t chroma_intra_matrix[64], uint16_t chroma_intra_matrix[64],
int hsample[3], int use_slices) int hsample[3], int use_slices, int matrices_differ)
{ {
int i, j, size; int i, j, size;
uint8_t *ptr; uint8_t *ptr;
if (m) { if (m) {
int matrix_count = 1 + !!memcmp(luma_intra_matrix, int matrix_count = 1 + matrices_differ;
chroma_intra_matrix,
sizeof(luma_intra_matrix[0]) * 64);
if (m->force_duplicated_matrix) if (m->force_duplicated_matrix)
matrix_count = 2; matrix_count = 2;
/* quant matrixes */ /* quant matrixes */
@@ -285,9 +283,7 @@ void ff_mjpeg_encode_picture_header(AVCodecContext *avctx, PutBitContext *pb,
const int lossless = !m; const int lossless = !m;
int hsample[4], vsample[4]; int hsample[4], vsample[4];
int components = 3 + (avctx->pix_fmt == AV_PIX_FMT_BGRA); int components = 3 + (avctx->pix_fmt == AV_PIX_FMT_BGRA);
int chroma_matrix = !!memcmp(luma_intra_matrix, int chroma_matrix;
chroma_intra_matrix,
sizeof(luma_intra_matrix[0])*64);
ff_mjpeg_init_hvsample(avctx, hsample, vsample); ff_mjpeg_init_hvsample(avctx, hsample, vsample);
@@ -299,9 +295,12 @@ void ff_mjpeg_encode_picture_header(AVCodecContext *avctx, PutBitContext *pb,
jpeg_put_comments(avctx, pb, frame); jpeg_put_comments(avctx, pb, frame);
chroma_matrix = !lossless && !!memcmp(luma_intra_matrix,
chroma_intra_matrix,
sizeof(luma_intra_matrix[0]) * 64);
jpeg_table_header(avctx, pb, m, intra_scantable, jpeg_table_header(avctx, pb, m, intra_scantable,
luma_intra_matrix, chroma_intra_matrix, hsample, luma_intra_matrix, chroma_intra_matrix, hsample,
use_slices); use_slices, chroma_matrix);
switch (avctx->codec_id) { switch (avctx->codec_id) {
case AV_CODEC_ID_MJPEG: put_marker(pb, SOF0 ); break; case AV_CODEC_ID_MJPEG: put_marker(pb, SOF0 ); break;