[MPlayer-dev-eng] [PATCH] "verbose" in libmpcodecs/pullup.c
Yue Shi Lai
ylai at users.sourceforge.net
Fri Apr 1 04:45:27 CEST 2005
D Richard Felker III schrieb:
> On Thu, Mar 31, 2005 at 02:44:14PM -0500, Yue Shi Lai wrote:
>
>>Hi,
>>
>>The line that checks the global variable "verbose" is implemented wrong
>>in libmpcodecs/pullup.c. The fact that "verbose" is a _signed_ variable
>>is disregarded, which results in the undesired increasing verbosity with
>>a negative value.
>
>
> it's not implemented wrong; verbose is a boolean flag. the way the
> field is declared is irrelevant.
Sorry, apparently you do not know what I am talking about. Maybe you
should have a look on vf_pullup.c first:
c->verbose = verbose;
The global verbose is definitively not boolean. And in the mean time,
maybe you also want to try run the filter using -really-quiet (which
sets the global verbose to a negative value).
So, pullup being extra loud, while the user sets -really-quiet, is this
irrelevant?
> if you want to document the api for
> pullup.c, feel free.. for now it's undocumented. :)
I am sorry: How are people supposed to document the API if your
statement is contradicted by the source code? By pure guessing?
More information about the MPlayer-dev-eng
mailing list