[MPlayer-cvslog] r20493 - in trunk: libass/Makefile libass/ass.c libass/ass.h libass/ass_library.c libass/ass_library.h libass/ass_mp.c libass/ass_mp.h libass/ass_render.c libass/ass_types.h libmpcodecs/vf_ass.c libmpcodecs/vf_vo.c libmpdemux/demux_mkv.c mplayer.c

Evgeniy Stepanov eugeni.stepanov at gmail.com
Sun Oct 29 00:19:00 CEST 2006


On Saturday 28 October 2006 20:09, Reimar Döffinger wrote:
> Hello,
>
> On Sat, Oct 28, 2006 at 05:07:18PM +0200, eugeni wrote:
> > Libass interface reworked:
> > - ass_instance_t renamed to ass_renderer_t
>
> renaming is a cosmetic change and _must_ _always be done seperately.

Right. Sorry. Should I re-commit ?

> > - ass_library_t introduced
> > - use of mplayer-specific global variables limited to ass_mp.c
>
> In general, do not apply unrelated things together, you are no longer
> the only one both caring about and being responsible for that code.

The second one is not a separate change, but a consequence of the first one. 
It could be applied as maybe 10 separate commits (one for each global 
variable), but I think it's not worth the effort.



More information about the MPlayer-cvslog mailing list