[MPlayer-cvslog] r27550 - in trunk: libmpdemux/demux_mkv.c mpcommon.c

[Diego Biurrun]

On Thu, Sep 11, 2008 at 10:46:15AM +0200, Reimar Doeffinger wrote:
> On Wed, Sep 10, 2008 at 04:03:48PM +0200, Diego Biurrun wrote:
> > > but only _in addition_ to some kind of abreviated "executive summary" of max. 4 lines
> > > that mentions the major what and whys.
> > 
> > I agree that there should be an executive summary, but this message does
> > actually have one:
> > 
> >   Revert bad changes to SSA/ASS subtitle packet format
> > 
> >   The following commits are reverted partially or completely:
> >   "a valid ASS line contains 9 ',' before actual text"
> >   "demux_mkv: output correctly formated ASS packets"
> >   "libass: add a new ass_process_data() to process demuxed subtitle packets"
> 
> Nope, that is only the "what" part.

Well, the "why" part takes longer to explain..

> For the long term, quoting the commit messages
> without the revision numbers is also a bit questionable.

I very much disagree.  I would still like to merge all the old branches
and the lost code and create a super-repo one day...

Diego