[FFmpeg-devel] [PATCH] Mark AVFilterPad[] compound literals as const.

Clément Bœsch ubitux at gmail.com
Sat Nov 5 15:21:49 CET 2011


On Sat, Nov 05, 2011 at 03:11:40PM +0100, Nicolas George wrote:
> Le quintidi 15 brumaire, an CCXX, Clément Bœsch a écrit :
> > > +    .inputs    = (const AVFilterPad[]) {{ .name            = "default",
> > >                                      .type            = AVMEDIA_TYPE_AUDIO,
> > >                                      .filter_samples  = filter_samples,
> > >                                      .min_perms       = AV_PERM_READ, },
> > >                                    { .name = NULL}},
> > 
> > Do you mind keeping the vertical alignment? :)
> 
> Usually, this is done in a separate patch.
> 
> And the lines will go beyond 80 columns. Better write it as:
> 
>     .inputs    = (const AVFilterPad[]) {
>         { .name            = "default",
> 	  .type            = AVMEDIA_TYPE_AUDIO,
> 	  .filter_samples  = filter_samples,
> 	  .min_perms       = AV_PERM_READ, },
> 	{ .name = NULL}
>     },
> 

I was thinking of just aligning the '=' after the .name attribute (line is
already changed anyway) and avoid a huge reindent patch.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111105/63f1e861/attachment.asc>


More information about the ffmpeg-devel mailing list