From: Martin Storsjö <martin@martin.st>
Date: Thu, 18 Oct 2012 07:23:12 +0000 (+0300)
Subject: gxf: Add a local copy of the relevant parts of the frame rate table
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=fc085c5b33a966ad893757ab1c4269bf829ce499;p=ffmpeg

gxf: Add a local copy of the relevant parts of the frame rate table

This avoids having to share this table across the library
boundaries.

This shared table used to be problematic, if always declaring
all exported data symbols with the dllimport attribute (even
while building that same library), since it needs to be a
link-time constant when it is used in AVCodec declarations
(in mpeg12enc.c).

Signed-off-by: Martin Storsjö <martin@martin.st>
---

diff --git a/libavformat/gxf.c b/libavformat/gxf.c
index 04d75bd4f2c..2593c0642b7 100644
--- a/libavformat/gxf.c
+++ b/libavformat/gxf.c
@@ -180,6 +180,18 @@ static void gxf_material_tags(AVIOContext *pb, int *len, struct gxf_stream_info
     }
 }
 
+static const AVRational frame_rate_tab[] = {
+    {   60,    1},
+    {60000, 1001},
+    {   50,    1},
+    {   30,    1},
+    {30000, 1001},
+    {   25,    1},
+    {   24,    1},
+    {24000, 1001},
+    {    0,    0},
+};
+
 /**
  * @brief convert fps tag value to AVRational fps
  * @param fps fps value from tag
@@ -187,7 +199,7 @@ static void gxf_material_tags(AVIOContext *pb, int *len, struct gxf_stream_info
  */
 static AVRational fps_tag2avr(int32_t fps) {
     if (fps < 1 || fps > 9) fps = 9;
-    return avpriv_frame_rate_tab[9 - fps]; // values have opposite order
+    return frame_rate_tab[fps - 1];
 }
 
 /**