[FFmpeg-cvslog] r25385 - trunk/libavformat/mpegenc.c

Diego Biurrun diego
Fri Oct 8 11:08:03 CEST 2010


On Thu, Oct 07, 2010 at 08:33:19PM +0200, Michael Niedermayer wrote:
> On Thu, Oct 07, 2010 at 02:17:10PM +0200, Diego Biurrun wrote:
> > On Thu, Oct 07, 2010 at 12:24:15PM +0200, Michael Niedermayer wrote:
> > > On Wed, Oct 06, 2010 at 08:16:20PM -0700, Alex Converse wrote:
> > > > On Wed, Oct 6, 2010 at 6:57 PM, michael <subversion at mplayerhq.hu> wrote:
> > > > >
> > > > > Log:
> > > > > Warn if muxing mpeg ps is attempted without a VBV buffer size.
> > > > >
> > > > > --- trunk/libavformat/mpegenc.c Thu Oct ?7 03:57:35 2010 ? ? ? ?(r25384)
> > > > > +++ trunk/libavformat/mpegenc.c Thu Oct ?7 03:57:39 2010 ? ? ? ?(r25385)
> > > > > @@ -367,8 +367,10 @@ static int mpeg_mux_init(AVFormatContext
> > > > > ? ? ? ? ? ? if (st->codec->rc_buffer_size)
> > > > > ? ? ? ? ? ? ? ? stream->max_buffer_size = 6*1024 + st->codec->rc_buffer_size/8;
> > > > > - ? ? ? ? ? ?else
> > > > > + ? ? ? ? ? ?else{
> > > > 
> > > > Just two days ago people asked you to put whitespace after keywords.
> > > 
> > > yes and iam not doing it, you should vote on me being leader accordingly.
> > 
> > The attitude you show when criticized...
> > 
> > > also IMHO, the one who said it loud enough for me to remember it at all is
> > > diego (on the phone ;) ) and he
> > > neither maintains C code nor is he contributing C code. And this is stuff
> > > that should be decided by the people falling in these 2 categories
> > 
> > Except of course for Stefano two days ago.  And who would dare to
> > criticize you when you immediately threaten to kill the critic or
> > yourself?
> 
> killing myself is my choice ;)
> and i didnt threten to kill any critic

Correct.  You skip the threats and proceed to kill them.

> > I'll also have to note that K&R style is mandated in the developer
> > guidelines.  But those guidelines are just something you use to
> > smack people over the head, not something you feel bound by
> > yourself.
> 
> Ill quote the policy for you, and id like to politely but sternly point out
> that people should _read_ the policy before accusing others about breaking it.
> 
> 
>    We refuse source indentation and other cosmetic changes if they are mixed
>    with functional changes, such commits will be rejected and removed. Every
>                                                                        ^^^^  
>    developer has his own indentation style, you should not change it. Of course
>    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>    if you (re)write something, you can use your own style, even though we would
>    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>    prefer if the indentation throughout FFmpeg was consistent (Many projects
>    force a given indentation style - we do not.). If you really need to make
>    indentation changes (try to avoid this), separate them strictly from real
>    changes.
> 
> Its you breaking this continously not me. You want things to be different
> and you always wanted things to be different but preaching everywhere that
> things are different doesnt make them so.

This paragraph was added by you without consultation and you break the
formatting of files that already use a consistent indentation style
without second thought.

> And the other part (not part of the policy) about style says that:
>     Indent size is 4.
>     The presentation is one inspired by 'indent -i4 -kr -nut'.
>     The TAB character is forbidden outside of Makefiles as is any
>     form of trailing whitespace. Commits containing either will be
>     rejected by the Subversion repository.
> 
> This says 'INSPIRED by 'indent -i4 -kr -nut'' it does not say follow the
> indent output to the last whitespace is mandatory

The "inspired" is a change unilaterally approved by yourself.

Diego



More information about the ffmpeg-cvslog mailing list