[FFmpeg-devel] [PATCH] cleanup image2pipe_{de}muxer fields
Pascal Massimino
pascal.massimino
Thu Oct 21 23:23:32 CEST 2010
Aurelien,
On Thu, Oct 21, 2010 at 2:00 PM, Aurelien Jacobs <aurel at gnuage.org> wrote:
> On Tue, Oct 19, 2010 at 07:15:44AM -0700, Pascal Massimino wrote:
> > Hi,
> >
> > $attached
> > cosmetic, but handy.
> >
> > [...]
> >
> > /* output */
> > #if CONFIG_IMAGE2_MUXER
> > AVOutputFormat image2_muxer = {
> > - "image2",
> > - NULL_IF_CONFIG_SMALL("image2 sequence"),
> > - "",
> > -
> "bmp,jpeg,jpg,ljpg,pam,pbm,pcx,pgm,pgmyuv,png,ppm,sgi,tga,tif,tiff,jp2",
> > - sizeof(VideoData),
> > - CODEC_ID_NONE,
> > - CODEC_ID_MJPEG,
> > - img_write_header,
> > - img_write_packet,
> > - NULL,
> > - .flags= AVFMT_NOTIMESTAMPS | AVFMT_NODIMENSIONS | AVFMT_NOFILE
> > + .name = "image2",
> > + .long_name = NULL_IF_CONFIG_SMALL("image2 sequence"),
>
> > + .mime_type = "",
>
> Seems useless.
>
> > + .extensions =
> "bmp,jpeg,jpg,ljpg,pam,pbm,pcx,pgm,pgmyuv,png,ppm,sgi,tga,tif,tiff,jp2",
> > + .priv_data_size = sizeof(VideoData),
>
> > + .audio_codec = CODEC_ID_NONE,
>
> useless.
>
> > + .video_codec = CODEC_ID_MJPEG,
> > + .write_header = write_header,
> > + .write_packet = write_packet,
> > + .flags = AVFMT_NOTIMESTAMPS | AVFMT_NODIMENSIONS | AVFMT_NOFILE
>
> Also you should use this opportunity to properly align those
> initialisations (all '=' on the same column).
>
> All those comments apply to all the stucts you modified.
>
>
as in $attached?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: img2.diff
Type: application/octet-stream
Size: 2872 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101021/1e179f1d/attachment.obj>
More information about the ffmpeg-devel
mailing list