On Wed, Aug 12, 2015 at 06:24:32PM +0530, Niklesh Lalwani wrote:
[...]
+#define BOTTOM_LEFT 1
+#define BOTTOM_CENTER 2
+#define BOTTOM_RIGHT3
+#define MIDDLE_LEFT 4
+#define MIDDLE_CENTER 5
+#define MIDDLE_RIGHT6
+#define TOP_LEFT7
+#define TOP_CENTER 8
On Wed, 12 Aug 2015 18:24:32 +0530
Niklesh Lalwani niklesh.lalw...@iitb.ac.in wrote:
From: Niklesh niklesh.lalw...@iitb.ac.in
As suggested, posting the combined patch with the fate changes.
The patch sets the default style in ASS from the default style
information present in the movtext
From: Niklesh niklesh.lalw...@iitb.ac.in
As suggested, posting the combined patch with the fate changes.
The patch sets the default style in ASS from the default style information
present in the movtext header.
Signed-off-by: Niklesh niklesh.lalw...@iitb.ac.in
---
libavcodec/movtextdec.c
From: Niklesh niklesh.lalw...@iitb.ac.in
Sets the default style in ASS from the default style information present in
movtext header
Signed-off-by: Niklesh niklesh.lalw...@iitb.ac.in
---
libavcodec/movtextdec.c | 82 ++---
1 file changed, 78
On Mon, 10 Aug 2015 01:18:58 +0530
Niklesh Lalwani niklesh.lalw...@iitb.ac.in wrote:
+
+typedef struct {
+char *font;
+int fontsize;
+int color;
+int back_color;
+int bold;
+int italic;
+int underline;
+int alignment;
+}
+
+typedef struct {
+char *font;
+int fontsize;
+int color;
+int back_color;
+int bold;
+int italic;
+int underline;
+int alignment;
+} Movtext_default;
+
This name is inconsistent. Use MovTextDefault.
Yes, makes sense. Updated patch
On Mon, Aug 10, 2015 at 01:18:58AM +0530, Niklesh Lalwani wrote:
+
+typedef struct {
+char *font;
+int fontsize;
+int color;
+int back_color;
+int bold;
+int italic;
+int underline;
+int alignment;
+} Movtext_default;
+
This