On 2014-10-11 19:59, Clément Bœsch wrote: > diff --git a/libavcodec/ass_split.h b/libavcodec/ass_split.h > index 06c1ce3..c912252 100644 > --- a/libavcodec/ass_split.h > +++ b/libavcodec/ass_split.h > @@ -41,13 +41,28 @@ typedef struct { > char *font_name; /**< font face (case sensitive) */ > int font_size; /**< font height */ > int primary_color; /**< color that a subtitle will normally appear in > */ > + int secondary_color; > + int outline_color; /**< color for outline in ASS, called tertiary in > SSA */ > int back_color; /**< color of the subtitle outline or shadow */ > int bold; /**< whether text is bold (1) or not (0) */ > int italic; /**< whether text is italic (1) or not (0) */ > int underline; /**< whether text is underlined (1) or not (0) */ > + int strikeout; > + float scalex; > + float scaley; > + float spacing; > + float angle; > + int border_style; > + float outline; > + float shadow; > int alignment; /**< position of the text (left, center, top...), > defined after the layout of the numpad > (1-3 sub, 4-6 mid, 7-9 top) */ > + int margin_l; > + int margin_r; > + int margin_v; > + int alpha_level; > + int encoding; > } ASSStyle; > > /** > @@ -58,6 +73,11 @@ typedef struct { > int start; /**< start time of the dialog in centiseconds */ > int end; /**< end time of the dialog in centiseconds */ > char *style; /**< name of the ASSStyle to use with this dialog */ > + char *name; > + int margin_l; > + int margin_r; > + int margin_v; > + char *effect; > char *text; /**< actual text which will be displayed as a subtitle, > can include style override control codes (see > ff_ass_split_override_codes()) */ >
You haven't put any comments describing the new fields (okay, you have done 1). I don't want to block the patch but it is better to comment things when they're added so that one doesn't go work on something else and then forget. Other than that, it looks fine.
signature.asc
Description: OpenPGP digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel