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

Clément Bœsch ubitux at gmail.com
Sat Nov 5 18:03:05 CET 2011


On Sat, Nov 05, 2011 at 03:35:03PM +0100, Reimar Döffinger wrote:
> On Sat, Nov 05, 2011 at 03:21:49PM +0100, Clément Bœsch wrote:
> > 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.
> 
> As you see that's a big bikeshed issue, it is additional work,
> and I couldn't decide on how to do it (personally I think it would
> be nicer to put .name on a separate line and reduce the amount of
> indentation).
> So, yes I do in fact mind.
[...]

I just thought you missed the align was broken and wanted to point it out.
Disregard my comment then.

-- 
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/8aeffafe/attachment.asc>


More information about the ffmpeg-devel mailing list