[FFmpeg-cvslog] r8912 - trunk/doc/ffmpeg-doc.texi

[Michael Niedermayer]

Hi

On Sun, May 06, 2007 at 02:06:55PM +0200, Diego Biurrun wrote:
> On Sun, May 06, 2007 at 01:41:42PM +0200, Michael Niedermayer wrote:
> > 
> > On Sun, May 06, 2007 at 12:48:41PM +0200, diego wrote:
> > > 
> > > Log:
> > > Add a note about maintaining alphabetical order to patch submission checklist.
> > 
> > [...]
> > >  @item
> > > +    Longer lists of things are usually kept in alphabetical order. 
> > 
> > this is irrelevant and if true, just so by chance
> > just because some list is long does not mean it should be in aphabetical order
> > similarely just because a list is short does not mean it should not be in
> > alphabetical order
> > 
> > > If you
> > > +    add an item, make sure to maintain alphabetical order.
> > 
> > a long enum or large struct doesnt have its members in alphabetical order
> > normally and even if it does its not possible to maintain this with public
> > stuff
> > 
> > so while i agree with the spirit of the change i dont agree at all with the
> > used wording
> > 
> > it rather should be something like
> > 
> > you should maintain alphabetical order in alphabetically ordered lists as
> > long as doing so does not break API/ABI compatibility
> 
> Perfect, commit it. :)

done

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070506/61a4b474/attachment.pgp>