[FFmpeg-devel] [PATCH 09/44] avformat/utils: Move ff_get_packet_palette() to rawutils.c

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Sat May 7 14:27:55 EEST 2022


ff_get_packet_palette() and ff_reshuffle_raw_rgb() belong together:
E.g. the former takes the return value of the latter as argument.
So move ff_get_packet_palette() to rawutils.c (which consists solely
of ff_reshuffle_raw_rgb()).
Also add a separate header for these two functions.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
---
 libavformat/avienc.c   |  1 +
 libavformat/internal.h | 27 --------------------
 libavformat/movenc.c   |  1 +
 libavformat/rawutils.c | 28 ++++++++++++++++++++-
 libavformat/rawutils.h | 56 ++++++++++++++++++++++++++++++++++++++++++
 libavformat/utils.c    | 24 ------------------
 6 files changed, 85 insertions(+), 52 deletions(-)
 create mode 100644 libavformat/rawutils.h

diff --git a/libavformat/avienc.c b/libavformat/avienc.c
index e36e88335f..2264241d57 100644
--- a/libavformat/avienc.c
+++ b/libavformat/avienc.c
@@ -28,6 +28,7 @@
 #include "config_components.h"
 #include "riff.h"
 #include "mpegts.h"
+#include "rawutils.h"
 #include "libavformat/avlanguage.h"
 #include "libavutil/avstring.h"
 #include "libavutil/avutil.h"
diff --git a/libavformat/internal.h b/libavformat/internal.h
index eb18277cea..da9abde637 100644
--- a/libavformat/internal.h
+++ b/libavformat/internal.h
@@ -901,33 +901,6 @@ int ff_parse_creation_time_metadata(AVFormatContext *s, int64_t *timestamp, int
  */
 int ff_standardize_creation_time(AVFormatContext *s);
 
-#define CONTAINS_PAL 2
-/**
- * Reshuffles the lines to use the user specified stride.
- *
- * @param ppkt input and output packet
- * @return negative error code or
- *         0 if no new packet was allocated
- *         non-zero if a new packet was allocated and ppkt has to be freed
- *         CONTAINS_PAL if in addition to a new packet the old contained a palette
- */
-int ff_reshuffle_raw_rgb(AVFormatContext *s, AVPacket **ppkt, AVCodecParameters *par, int expected_stride);
-
-/**
- * Retrieves the palette from a packet, either from side data, or
- * appended to the video data in the packet itself (raw video only).
- * It is commonly used after a call to ff_reshuffle_raw_rgb().
- *
- * Use 0 for the ret parameter to check for side data only.
- *
- * @param pkt pointer to packet before calling ff_reshuffle_raw_rgb()
- * @param ret return value from ff_reshuffle_raw_rgb(), or 0
- * @param palette pointer to palette buffer
- * @return negative error code or
- *         1 if the packet has a palette, else 0
- */
-int ff_get_packet_palette(AVFormatContext *s, AVPacket *pkt, int ret, uint32_t *palette);
-
 struct AVBPrint;
 /**
  * Finalize buf into extradata and set its size appropriately.
diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index d32328d40c..c720fd742d 100644
--- a/libavformat/movenc.c
+++ b/libavformat/movenc.c
@@ -62,6 +62,7 @@
 #include "mov_chan.h"
 #include "movenc_ttml.h"
 #include "mux.h"
+#include "rawutils.h"
 #include "ttmlenc.h"
 #include "version.h"
 #include "vpcc.h"
diff --git a/libavformat/rawutils.c b/libavformat/rawutils.c
index 996412adb9..e44c946d47 100644
--- a/libavformat/rawutils.c
+++ b/libavformat/rawutils.c
@@ -19,8 +19,10 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
+#include "libavutil/intreadwrite.h"
+#include "libavcodec/packet.h"
 #include "avformat.h"
-#include "internal.h"
+#include "rawutils.h"
 
 int ff_reshuffle_raw_rgb(AVFormatContext *s, AVPacket **ppkt, AVCodecParameters *par, int expected_stride)
 {
@@ -65,3 +67,27 @@ fail:
 
     return ret;
 }
+
+int ff_get_packet_palette(AVFormatContext *s, AVPacket *pkt, int ret, uint32_t *palette)
+{
+    uint8_t *side_data;
+    size_t size;
+
+    side_data = av_packet_get_side_data(pkt, AV_PKT_DATA_PALETTE, &size);
+    if (side_data) {
+        if (size != AVPALETTE_SIZE) {
+            av_log(s, AV_LOG_ERROR, "Invalid palette side data\n");
+            return AVERROR_INVALIDDATA;
+        }
+        memcpy(palette, side_data, AVPALETTE_SIZE);
+        return 1;
+    }
+
+    if (ret == CONTAINS_PAL) {
+        for (int i = 0; i < AVPALETTE_COUNT; i++)
+            palette[i] = AV_RL32(pkt->data + pkt->size - AVPALETTE_SIZE + i*4);
+        return 1;
+    }
+
+    return 0;
+}
diff --git a/libavformat/rawutils.h b/libavformat/rawutils.h
new file mode 100644
index 0000000000..a91123412b
--- /dev/null
+++ b/libavformat/rawutils.h
@@ -0,0 +1,56 @@
+/*
+ * copyright (c) 2001 Fabrice Bellard
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * FFmpeg is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#ifndef AVFORMAT_RAWUTILS_H
+#define AVFORMAT_RAWUTILS_H
+
+#include <stdint.h>
+#include "libavcodec/codec_par.h"
+#include "libavcodec/packet.h"
+#include "avformat.h"
+
+#define CONTAINS_PAL 2
+/**
+ * Reshuffles the lines to use the user specified stride.
+ *
+ * @param ppkt input and output packet
+ * @return negative error code or
+ *         0 if no new packet was allocated
+ *         non-zero if a new packet was allocated and ppkt has to be freed
+ *         CONTAINS_PAL if in addition to a new packet the old contained a palette
+ */
+int ff_reshuffle_raw_rgb(AVFormatContext *s, AVPacket **ppkt, AVCodecParameters *par, int expected_stride);
+
+/**
+ * Retrieves the palette from a packet, either from side data, or
+ * appended to the video data in the packet itself (raw video only).
+ * It is commonly used after a call to ff_reshuffle_raw_rgb().
+ *
+ * Use 0 for the ret parameter to check for side data only.
+ *
+ * @param pkt pointer to packet before calling ff_reshuffle_raw_rgb()
+ * @param ret return value from ff_reshuffle_raw_rgb(), or 0
+ * @param palette pointer to palette buffer
+ * @return negative error code or
+ *         1 if the packet has a palette, else 0
+ */
+int ff_get_packet_palette(AVFormatContext *s, AVPacket *pkt, int ret, uint32_t *palette);
+
+#endif /* AVFORMAT_RAWUTILS_H */
diff --git a/libavformat/utils.c b/libavformat/utils.c
index b53e2aff0c..cc5507a19e 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -1715,30 +1715,6 @@ int ff_standardize_creation_time(AVFormatContext *s)
     return ret;
 }
 
-int ff_get_packet_palette(AVFormatContext *s, AVPacket *pkt, int ret, uint32_t *palette)
-{
-    uint8_t *side_data;
-    size_t size;
-
-    side_data = av_packet_get_side_data(pkt, AV_PKT_DATA_PALETTE, &size);
-    if (side_data) {
-        if (size != AVPALETTE_SIZE) {
-            av_log(s, AV_LOG_ERROR, "Invalid palette side data\n");
-            return AVERROR_INVALIDDATA;
-        }
-        memcpy(palette, side_data, AVPALETTE_SIZE);
-        return 1;
-    }
-
-    if (ret == CONTAINS_PAL) {
-        for (int i = 0; i < AVPALETTE_COUNT; i++)
-            palette[i] = AV_RL32(pkt->data + pkt->size - AVPALETTE_SIZE + i*4);
-        return 1;
-    }
-
-    return 0;
-}
-
 int ff_bprint_to_codecpar_extradata(AVCodecParameters *par, struct AVBPrint *buf)
 {
     int ret;
-- 
2.32.0



More information about the ffmpeg-devel mailing list