[FFmpeg-devel] [PATCH 2/2] movtextenc.c: Support for Bold, Italic and Underlined styles

Niklesh Lalwani niklesh.lalwani at iitb.ac.in
Fri Jun 26 22:25:03 CEST 2015

On 27-Jun-2015 12:36 AM, "Philip Langdale" <philipl at overt.org> wrote:
> On 2015-06-26 09:48, Niklesh Lalwani wrote:
>> The output produced here was actually different from the expected. Hence
>> the test failed. I produced ASS file for both the original file and the
>> encoded file. The ass file produced from the encoded mp4 file seemed to
>> dropping all text after \N
>> After a lot of digging, I found out that while appending '\n' into the
>> buffer in encoding, it is to be considered as a single character and not
>> Making this change in movtextenc.c solved the issue and now I have
>> ASS outputs for both the files.
>> However, the test still fails even though I have exact same ASS files.
>> PS - there is no style information associated here as far as I can see.
>> Please correct me if I'm going wrong here.
> This test produces another mp4 file - not just an ASS subtitle (which is
> what the other movtextenc test does. So something else you've done has
> altered the output - perhaps through metadata or adding a box that wasn't
> there before, etc.

You are right. My movenc.c file was different ( I didn't undo the commit to
set the subtitle dimensions in my working branch which I submitted here
some time back)
Got the latest movenc.c file from git, and the test doesn't give any errors
now ( after making the change in movtextenc.c which I mentioned before)

Updated patch attached.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-movtextenc.c-Support-for-Bold-Italic-and-Underlined-.patch
Type: application/octet-stream
Size: 9144 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150627/989c13f8/attachment.obj>

More information about the ffmpeg-devel mailing list