On 19/02/13 07:57, Reinhard Tartler wrote:
> On Mon, Feb 18, 2013 at 10:50 PM, Luca Barbato <[email protected]> wrote:
>> On 18/02/13 21:04, Reinhard Tartler wrote:
>>
>> I'm not sure it is _that_ compelling.
>>
>>> +                           NB: Cannot be combined with --target-exec.
>>
>> This line could go in the documentation (malloc-fill is debatable btw)
>> but we shouldn't force this, it makes the whole feature less useful.
> 
> I'm not sure if I grasp your review in full. I understand that you
> object to having that line in the configure output, but suggest to
> place it somewhere in doc/developer.texi instead. Is this correct?
> 

Instead of hardwiring valgrind this way I'd:

- document an useful command line
- have the suppression file in documentation or contrib

If you feel that we should absolutely provide a configure option, I
would make it a little general (I know people are using pin and other
analyzers/binary-patchers).

lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to