|
From: Eliot M. <mo...@cs...> - 2014-09-24 13:51:46
|
On 9/24/2014 7:39 AM, Eliot Moss wrote: > On 9/24/2014 5:31 AM, Skarakis, Konstantinos wrote: Ok -- I decided to just give it a try in a copy of valgrind updated to svn head. The issue is that I was temporarily sticking a null character into the format string, yet the format string was declared somewhere else as const. That declaration was righteous, and my temporary insertion of a null I admit was a hack. I was already copying the relevant part of the format string out into a temporary buffer, so the actual adjustments needed were small. Once I made those changes, it compiles and passes check tests for me (on a linux amd64 system). I attach the updated patch. I'll see about submitting this, though there also needs to be an update to a section of the manual for a complete patch to make sense for the distro. Regards -- Eliot Moss |