[FFmpeg-devel] [PATCH] Add support for sndio to libavdevice

Aurelien Jacobs aurel
Tue Aug 10 18:46:57 CEST 2010


On Tue, Aug 10, 2010 at 02:15:12AM +0200, Diego Biurrun wrote:
> On Mon, Aug 09, 2010 at 07:01:50PM +0200, Aurelien Jacobs wrote:
> > On Sun, Aug 08, 2010 at 10:19:41PM +0100, M?ns Rullg?rd wrote:
> > > Diego Biurrun <diego at biurrun.de> writes:
> > > 
> > > > On Thu, Aug 05, 2010 at 08:21:21AM +0100, M?ns Rullg?rd wrote:
> > > >> Diego Biurrun <diego at biurrun.de> writes:
> > > >> 
> > > >> > On Mon, Aug 02, 2010 at 07:42:28PM -0400, Brad wrote:
> > > >> >> 
> > > >> >> Please provide any feedback.
> > > >> >
> > > >> > Here you go..
> > > >> >
> > > >> >> --- Changelog	(revision 24666)
> > > >> >> +++ Changelog	(working copy)
> > > >> >> @@ -27,9 +27,9 @@
> > > >> >>  - SubRip subtitle file muxer and demuxer
> > > >> >>  - Chinese AVS encoding via libxavs
> > > >> >>  - ffprobe -show_packets option added
> > > >> >> +- sndio support for playback and record
> > > >> >>  
> > > >> >>  
> > > >> >> -
> > > >> >>  version 0.6:
> > > >> >
> > > >> > The empty line was there on purpose.
> > > >> 
> > > >> Care to tell us _what_ that purpose is?
> > > >
> > > > Readability, what else?
> > > 
> > > I fail to see how 3 blank lines are more readable than 2.
> > 
> > Agree. To me, 2 blank lines are more readable than 3.
> > I always wondered why those 3 lines were added. I thought it was to
> > comply to some stupid "standard", but certainly not for readability.
> 
> 3 lines are simply a larger and clearer offset than 2.

And 4 lines are a larger and clearer offset than 3...

> > Moreover, many patch senders tend to naturally remove the third line,
> > only to receive a laconic email from diego asking to resend the patch
> > without this change. So this additionnal line adds some useless
> > maintenance and patch submission burden.
> 
> I hold a bet that the same will happen with 2 lines, but we shall find
> out...

I don't remember about this happening before the switch to 3 lines.
Anyway, let's find out.

> > So I propose we just switch back to 2 lines, and end up this madness.
> 
> Go right ahead, but please consistently fix the complete file.

Sure. Done.

Aurel



More information about the ffmpeg-devel mailing list