[Ffmpeg-cvslog] r8410 - trunk/doc/ffmpeg-doc.texi
The Wanderer
inverseparadox
Thu Mar 15 22:42:03 CET 2007
michael wrote:
> Author: michael
> Date: Thu Mar 15 13:18:47 2007
> New Revision: 8410
>
> Modified:
> trunk/doc/ffmpeg-doc.texi
>
> Log:
> add a entry to the policy about fixing warnings (i think this is common sense but if anydeveloper disagrees, just say so
> and ill revert it and we can disscuss it on ffmpeg-dev)
> + at item
> + Do not change code to hide warnings without ensuring that the underlaying
> + logic is correct and thus the warning was inappropriate
Minor nit: "underlying". (The specific reasons why elude me for the
moment, but I believe it's something to do with the fact that "underlay"
would be past tense but we want a sort of 'continuing present' tense
here.)
I had some fairly extensive proofreading comments on your other commits
to this file today, but Diego seems to have already caught almost all of
them. (Presumably he will get the ones in the MPlayer copy as well.)
--
The Wanderer
Warning: Simply because I argue an issue does not mean I agree with any
side of it.
Secrecy is the beginning of tyranny.
More information about the ffmpeg-cvslog
mailing list