[FFmpeg-soc] [soc]: r225 - matroska/matroskaenc.c

conrad subversion at mplayerhq.hu
Sun Jun 3 03:39:24 CEST 2007


Author: conrad
Date: Sun Jun  3 03:39:24 2007
New Revision: 225

Log:
Get rid of put_ebml_utf8(); the error checking that would have distinguished it from put_ebml_string() belongs elsewhere in lavf.

Modified:
   matroska/matroskaenc.c

Modified: matroska/matroskaenc.c
==============================================================================
--- matroska/matroskaenc.c	(original)
+++ matroska/matroskaenc.c	Sun Jun  3 03:39:24 2007
@@ -75,17 +75,11 @@ static void put_ebml_binary(ByteIOContex
     put_buffer(pb, buf, size);
 }
 
-// XXX: should we do any special checking for valid strings for these 2 functions?
 static void put_ebml_string(ByteIOContext *pb, unsigned int elementid, char *str)
 {
     put_ebml_binary(pb, elementid, str, strlen(str));
 }
 
-static void put_ebml_utf8(ByteIOContext *pb, unsigned int elementid, char *str)
-{
-    put_ebml_binary(pb, elementid, str, strlen(str));
-}
-
 static offset_t start_ebml_master(ByteIOContext *pb, unsigned int elementid)
 {
     put_ebml_id(pb, elementid);
@@ -127,11 +121,11 @@ static int mkv_write_header(AVFormatCont
     segment_info = start_ebml_master(pb, MATROSKA_ID_INFO);
     put_ebml_uint(pb, MATROSKA_ID_TIMECODESCALE, 1000000);
     if (strlen(s->title))
-        put_ebml_utf8(pb, MATROSKA_ID_TITLE, s->title);
+        put_ebml_string(pb, MATROSKA_ID_TITLE, s->title);
     if (!(s->streams[0]->codec->flags & CODEC_FLAG_BITEXACT)) {
-        put_ebml_utf8(pb, MATROSKA_ID_MUXINGAPP, LIBAVFORMAT_IDENT);
+        put_ebml_string(pb, MATROSKA_ID_MUXINGAPP, LIBAVFORMAT_IDENT);
         // XXX: both are required; something better for writing app?
-        put_ebml_utf8(pb, MATROSKA_ID_WRITINGAPP, LIBAVFORMAT_IDENT);
+        put_ebml_string(pb, MATROSKA_ID_WRITINGAPP, LIBAVFORMAT_IDENT);
     }
     // XXX: segment UID and duration
     end_ebml_master(pb, segment_info);



More information about the FFmpeg-soc mailing list