[FFmpeg-devel] PATCH watermark + RGB32 input format stream
Michael Niedermayer
michaelni
Mon May 5 00:26:28 CEST 2008
On Sun, May 04, 2008 at 10:59:03PM +0200, Marcus Engene wrote:
> Michael Niedermayer wrote:
> > On Sun, May 04, 2008 at 09:53:02PM +0200, Marcus Engene wrote:
> >
> >> Hi,
> >>
> >> watermark acts up if the instream is something that offers watermark RGB32
> >> (for example qtrle)
> >>
> >> I don't know if the magnificent optimization the attached patch disables
> >> ever worked or if it was causing trouble after patches to make watermark
> >> honor recent api changes.
> >>
> >> Best regards,
> >> Marcus
> >>
> >>
> >
> >
> >> --- watermark.c.org 2008-05-04 21:06:56.000000000 +0200
> >> +++ watermark.c 2008-05-04 21:11:33.000000000 +0200
> >> @@ -208,33 +208,31 @@
> >> int thrG = ci->thrG;
> >> int thrB = ci->thrB;
> >>
> >> - if (pix_fmt != PIX_FMT_RGB32) {
> >> - int size;
> >> + int size;
> >>
> >> - size = avpicture_get_size(PIX_FMT_RGB32, src_width, src_height);
> >> - buf = av_malloc(size);
> >> + size = avpicture_get_size(PIX_FMT_RGB32, src_width, src_height);
> >> + buf = av_malloc(size);
> >>
> >
> > vhook is deprecated, i will not review patches to it except moving the
> > filters to avfilter.
> > and this is full of cosmetics
> >
> > [...]
> I removed a couple of if() - would you prefer to have the indentation
> looking like a French car from the 50s?
Please read the ffmpeg docs they explain the relation between cosmetics,
functional changes and patches.
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080505/8b1c6c77/attachment.pgp>
More information about the ffmpeg-devel
mailing list