diff mbox series

[FFmpeg-devel] avformat/jvdec: Make sizeof(JVFrame) smaller to save memory

Message ID AM7PR03MB66602DA1CE4887FFA7CBD20F8FA39@AM7PR03MB6660.eurprd03.prod.outlook.com
State Accepted
Commit 2882286a08c20fbb50bb55e16c82cbe6fdbfd565
Headers show
Series [FFmpeg-devel] avformat/jvdec: Make sizeof(JVFrame) smaller to save memory
Related show

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished

Commit Message

Andreas Rheinhardt Sept. 23, 2021, 4:29 p.m. UTC
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavformat/jvdec.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Peter Ross Sept. 23, 2021, 11:59 p.m. UTC | #1
On Thu, Sep 23, 2021 at 06:29:35PM +0200, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  libavformat/jvdec.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/jvdec.c b/libavformat/jvdec.c
> index 8f21ea1893..93569c87df 100644
> --- a/libavformat/jvdec.c
> +++ b/libavformat/jvdec.c
> @@ -36,8 +36,8 @@
>  typedef struct JVFrame {
>      int audio_size;    /**< audio packet size (bytes) */
>      int video_size;    /**< video packet size (bytes) */
> -    int palette_size;  /**< palette size (bytes) */
> -    int video_type;    /**< per-frame video compression type */
> +    uint16_t palette_size;  /**< palette size (bytes) */
> +    uint8_t video_type;     /**< per-frame video compression type */
>  } JVFrame;

ok

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
diff mbox series

Patch

diff --git a/libavformat/jvdec.c b/libavformat/jvdec.c
index 8f21ea1893..93569c87df 100644
--- a/libavformat/jvdec.c
+++ b/libavformat/jvdec.c
@@ -36,8 +36,8 @@ 
 typedef struct JVFrame {
     int audio_size;    /**< audio packet size (bytes) */
     int video_size;    /**< video packet size (bytes) */
-    int palette_size;  /**< palette size (bytes) */
-    int video_type;    /**< per-frame video compression type */
+    uint16_t palette_size;  /**< palette size (bytes) */
+    uint8_t video_type;     /**< per-frame video compression type */
 } JVFrame;
 
 typedef struct JVDemuxContext {