[Ffmpeg-devel] [PATCH] grabbing with V4L
Luca Abeni
lucabe72
Fri Feb 17 11:40:00 CET 2006
Hi Flavio,
On Wed, 2006-02-15 at 14:34 +0100, Luca Abeni wrote:
[...]
> > The patch I've sent didn't create a single new line, it only moves some
> > image settings initialization to outside the if statement. It was also
> > not my idea, I borrowed from mplayer ;)
> Yes, looking at the patch I agree that it should not create problems.
> I'll test it tomorrow, just as a confirmation.
I tested your patch, and verified that it does not create any regression.
Uhmm... I still do not understand why it is needed by your webcam driver
but the "regular" bttv driver does not need it...
Anyway, I think your patch is good because it permits to avoid some code
duplication (in fact, after applying your patch all the code around line
212 becomes useless). So, I went ahead and I removed some code that (I
think) is obsoleted by your patch (since desired_palette is already set
by previous code).
Does the attached patch (your patch + some useless code removal) work
for you?
Thanks,
Luca
--
_____________________________________________________________________________
Copy this in your signature, if you think it is important:
N O W A R ! ! !
--
Email.it, the professional e-mail, gratis per te: http://www.email.it/f
Sponsor:
Per i progetti che rimandi da tempo, Findomestic ti offre la soluzione ideale per te, con semplicit? e senza anticipi! Clicca e scopri come
Clicca qui: http://adv.email.it/cgi-bin/foclick.cgi?mid=3936&d=17-2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: grab-pix-format.diff
Type: text/x-patch
Size: 3899 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060217/468bec83/attachment.bin>
More information about the ffmpeg-devel
mailing list