[FFmpeg-devel] [PATCH/RFC] Make dvvideo encoder less lenient wrt resolution and pixel format

Tomas Härdin tomas.hardin
Fri Dec 11 16:25:48 CET 2009


On Fri, 2009-12-11 at 15:53 +0100, Diego Biurrun wrote:
> On Fri, Dec 11, 2009 at 01:42:46PM +0100, Tomas H?rdin wrote:
> > On Fri, 2009-12-11 at 13:29 +0100, Michael Niedermayer wrote:
> > > 
> > > retry without putting newlines in the middle of patches
> > 
> > Odd, I made sure to paste the lines correctly. Probably Evolution doing
> > things wrong. Attaching the patch to this mail instead.
> 
> OK, now please retry with K&R coding style, i.e.
> 
> > --- libavcodec/dv.c	(revision 20797)
> > +++ libavcodec/dv.c	(working copy)
> > @@ -398,6 +398,15 @@
> >  
> > +static av_cold int dvvideo_init_encoder(AVCodecContext *avctx) {
> 
> { on the next line for function declarations
> 
> > +    if(!ff_dv_codec_profile(avctx)) {
> 
> if (
> 
> > +        av_log(avctx, AV_LOG_ERROR, "Found no DV profile for %ix%i %s video\n", avctx->width, avctx->height, avcodec_get_pix_fmt_name(avctx->pix_fmt));
> 
> Break this overly long line.
> 
> Diego

Done. I guess I've been coding Java a bit too long :)
I went with a line break and single 4-space tab to break up the av_log()
call since I couldn't find any systematic indentation for such cases (I
saw 0, 4, 5 and 8 spaces).

New patch attached.

/Tomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dv.diff
Type: text/x-patch
Size: 980 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091211/60e69c01/attachment.bin>



More information about the ffmpeg-devel mailing list