On 17/03/13 17:43, Diego Biurrun wrote: > On Sun, Mar 17, 2013 at 07:59:35AM +0100, Reinhard Tartler wrote: >> >> --- a/configure >> +++ b/configure >> @@ -2155,6 +2155,10 @@ case "$toolchain" in >> add_cflags -fthread-sanitizer >> add_ldflags -fthread-sanitizer >> ;; >> + valgrind-memcheck) > > That valgrind is a memory checker is well-known. I would prefer a name > that suggests more of a combination with FATE, which is what the option > does, such as "fate-valgrind" or "valgrind-fate" or just plain "valgrind".
memcheck is the name of the tool in use. > Why not just add the args to target_exec? so you can override the valgrind and keep the command line. > It would seem that fate.texi is a better location for this note. if we have a toolchain section it should stay there, an hint about it would do. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
