Re: svn commit: r247960 - head/cddl/contrib/opensolaris/tools/ctf/cvt

2013-04-19 Thread Ulrich Spörlein
On Thu, 2013-03-07 at 20:45:20 -0500, Eitan Adler wrote: On 7 March 2013 18:00, Dimitry Andric d...@freebsd.org wrote: On 2013-03-07 23:29, Andriy Gapon wrote: ... I was not really kidding when I said that this change, as is, is a nop: ... + bname =

svn commit: r247960 - head/cddl/contrib/opensolaris/tools/ctf/cvt

2013-03-07 Thread Dimitry Andric
Author: dim Date: Thu Mar 7 22:16:35 2013 New Revision: 247960 URL: http://svnweb.freebsd.org/changeset/base/247960 Log: Make ctfconvert work correctly on clang-compiled object files. Clang puts the full original source filename in the STT_FILE entry of the ELF symbol table, while gcc

Re: svn commit: r247960 - head/cddl/contrib/opensolaris/tools/ctf/cvt

2013-03-07 Thread Andriy Gapon
on 08/03/2013 00:16 Dimitry Andric said the following: Author: dim Date: Thu Mar 7 22:16:35 2013 New Revision: 247960 URL: http://svnweb.freebsd.org/changeset/base/247960 Log: Make ctfconvert work correctly on clang-compiled object files. Clang puts the full original source filename

Re: svn commit: r247960 - head/cddl/contrib/opensolaris/tools/ctf/cvt

2013-03-07 Thread Dimitry Andric
On 2013-03-07 23:29, Andriy Gapon wrote: ... I was not really kidding when I said that this change, as is, is a nop: ... + bname = strrchr(match.iim_name, '/'); + bname = bname == NULL ? match.iim_name : bname + 1; It would probably make sense to

Re: svn commit: r247960 - head/cddl/contrib/opensolaris/tools/ctf/cvt

2013-03-07 Thread Eitan Adler
On 7 March 2013 18:00, Dimitry Andric d...@freebsd.org wrote: On 2013-03-07 23:29, Andriy Gapon wrote: ... I was not really kidding when I said that this change, as is, is a nop: ... + bname = strrchr(match.iim_name, '/'); + bname = bname ==