[FFmpeg-devel] [PATCH] vf_subtitles: add line_spacing option

[Clément Bœsch]

On Tue, May 21, 2013 at 10:52:38PM +0200, Michael Niedermayer wrote:
[...]
> > > A user on #ffmpeg had this problem:
> > > http://oi40.tinypic.com/e0pgkl.jpg . Is there another useful feature
> > > missing ?
> > > 
> > 
> > I don't know, but it seems there are quite some configuration settings
> > that could be set:
> > 
> >     % grep 'void ass_set_' /usr/include/ass/ass.h
> >     void ass_set_fonts_dir(ASS_Library *priv, const char *fonts_dir);
> >     void ass_set_extract_fonts(ASS_Library *priv, int extract);
> >     void ass_set_style_overrides(ASS_Library *priv, char **list);
> >     void ass_set_message_cb(ASS_Library *priv, void (*msg_cb)
> >     void ass_set_frame_size(ASS_Renderer *priv, int w, int h);
> >     void ass_set_shaper(ASS_Renderer *priv, ASS_ShapingLevel level);
> >     void ass_set_margins(ASS_Renderer *priv, int t, int b, int l, int r);
> >     void ass_set_use_margins(ASS_Renderer *priv, int use);
> >     void ass_set_aspect_ratio(ASS_Renderer *priv, double dar, double sar);
> >     void ass_set_font_scale(ASS_Renderer *priv, double font_scale);
> >     void ass_set_hinting(ASS_Renderer *priv, ASS_Hinting ht);
> >     void ass_set_line_spacing(ASS_Renderer *priv, double line_spacing);
> >     void ass_set_line_position(ASS_Renderer *priv, double line_position);
> >     void ass_set_fonts(ASS_Renderer *priv, const char *default_font,
> >     void ass_set_cache_limits(ASS_Renderer *priv, int glyph_max,
> > 
> > How relevant they are, I don't know. The patch in itself is fine, It's
> > just too bad not to port a bunch of others settings.
> 
> if the patch is fine, shouldnt it be applied ? or do you want me
> to apply it ?
> 

I think Jean wanted to send a new version after Nicolas' comment.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130521/735205bd/attachment.asc>