Commit 81efc03f authored by David Conrad's avatar David Conrad

Get rid of put_ebml_utf8(); the error checking that would have distinguished...

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

Originally committed as revision 10297 to svn://svn.ffmpeg.org/ffmpeg/trunk
parent dbf653f7
...@@ -75,17 +75,11 @@ static void put_ebml_binary(ByteIOContext *pb, unsigned int elementid, ...@@ -75,17 +75,11 @@ static void put_ebml_binary(ByteIOContext *pb, unsigned int elementid,
put_buffer(pb, buf, size); 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) static void put_ebml_string(ByteIOContext *pb, unsigned int elementid, char *str)
{ {
put_ebml_binary(pb, elementid, str, strlen(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) static offset_t start_ebml_master(ByteIOContext *pb, unsigned int elementid)
{ {
put_ebml_id(pb, elementid); put_ebml_id(pb, elementid);
...@@ -127,11 +121,11 @@ static int mkv_write_header(AVFormatContext *s) ...@@ -127,11 +121,11 @@ static int mkv_write_header(AVFormatContext *s)
segment_info = start_ebml_master(pb, MATROSKA_ID_INFO); segment_info = start_ebml_master(pb, MATROSKA_ID_INFO);
put_ebml_uint(pb, MATROSKA_ID_TIMECODESCALE, 1000000); put_ebml_uint(pb, MATROSKA_ID_TIMECODESCALE, 1000000);
if (strlen(s->title)) 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)) { 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? // 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 // XXX: segment UID and duration
end_ebml_master(pb, segment_info); end_ebml_master(pb, segment_info);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment