Re: [FFmpeg-devel] [PATCH]Add a closing CRLF to Webm-manifest files

2014-09-11 Thread Nicolas George
of an XML file seems like the right thing to do. > Subject: Re: [FFmpeg-devel] [PATCH]Add a closing CRLF to Webm-manifest files > +avio_printf(s->pb, "\n"); Small nit on the commit message: \n is just LF, CRLF would be \r\n. Just LF should be fine with XML, especially if the

Re: [FFmpeg-devel] [PATCH]Add a closing CRLF to Webm-manifest files

2014-09-11 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > Attached patch fixes fate-webm-dash-manifest on aix. This is the failure: $ gmake SAMPLES=fate-suite/ fate-webm-dash-manifest TESTwebm-dash-manifest diff: Missing newline at the end of file tests/data/fate/webm-dash-manifest. --- ./tests/ref/fate/webm-da