This is the mail archive of the
mailing list for the elfutils project.
Re: [patch v7 3/5] x86* unwinder: libdwfl/
- From: Mark Wielaard <mjw at redhat dot com>
- To: elfutils-devel at lists dot fedorahosted dot org
- Date: Mon, 28 Oct 2013 11:13:38 +0100
- Subject: Re: [patch v7 3/5] x86* unwinder: libdwfl/
On Sun, 2013-10-27 at 15:29 +0100, Jan Kratochvil wrote:
> Valgrind would always warn if the application did not set it. Even if the
> appplication never used it. It would warn on lines:
> if (! process->callbacks->set_initial_registers (thread,
> process->callbacks->thread_detach (thread, thread->callbacks_arg);
> as CALLBACKS_ARG is passed there by value, not by reference.
BTW. If you configure --with-valgrind then make check runs all tests
under valgrind. (make distcheck does the same, but takes a very long
time since it runs the tests multiple times.)
I don't think valgrind memcheck would warn in the above case for passing
around "undefined values". It only warns when those undefined bits are
actually used in a condition or end up being passed as syscall arguments
it knows should have defined values.
> Sorry but you really should be on the second line of ChangeLog. You wrote
> some parts of code and I already do not agree with various changes we made.
> I do not mind but all together it is not authored only by me.
Sure, you can use my name on the ChangeLog entries if you want and/or
add my Signed-off-by line on the commit. I do approve of the code and