[FFmpeg-devel] [PATCH] Fix some typos in avfilter.h

Michael Niedermayer michaelni
Sun Aug 24 03:07:11 CEST 2008


On Sun, Aug 24, 2008 at 02:56:42AM +0200, Michael Niedermayer wrote:
> On Sat, Aug 23, 2008 at 11:23:45PM +0200, Stefano Sabatini wrote:
> > Hi, as in subject.
> > 
> > I'm not sure for the second hunk.
> 
> So why do you include it?
> 
> patches arent being submitted as a test for them being correct, patches
> are submitted after the author is sure they are correct.
> 
> Your doxygen patches drive me nearly nuts, could you please either
> A. refrain from posting incorrect&unverified changes to doxygen stuff
> B. check your changes against the code
> 
> I think you did not ever post a change to a doxygen comment that was not
> half wrong and that i could not have done myself correctly in less time
> than it took me to review yours.

btw in case anyone wonders
from the 3 hunks
the first fixes a missing char typo and likely correct
the second is unneeded
and the third is outright wrong and could lead serious confusion
(thats all IMHO of course)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- 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/20080824/8960d35d/attachment.pgp>



More information about the ffmpeg-devel mailing list