On 6 Sep 2012, at 02:42, Gennady Proskurin <gpr...@mail.ru> wrote: > On Thu, Sep 06, 2012 at 03:19:49AM +0000, Rui Paulo wrote: >> Author: rpaulo >> Date: Thu Sep 6 03:19:48 2012 >> New Revision: 240156 >> URL: http://svn.freebsd.org/changeset/base/240156 >> >> Log: >> Add support for demangling C++ symbols. This requires linking libproc with >> libc++rt/libsupc++. >> >> Discussed with: theraven >> >> Modified: >> head/lib/libproc/Makefile >> head/lib/libproc/proc_sym.c >> >> Modified: head/lib/libproc/Makefile >> ============================================================================== >> --- head/lib/libproc/Makefile Thu Sep 6 02:07:58 2012 >> (r240155) >> +++ head/lib/libproc/Makefile Thu Sep 6 03:19:48 2012 >> (r240156) >> @@ -1,5 +1,7 @@ >> # $FreeBSD$ >> >> +.include <bsd.own.mk> >> + >> LIB= proc >> >> SRCS= proc_bkpt.c \ >> @@ -13,6 +15,14 @@ INCS= libproc.h >> >> CFLAGS+= -I${.CURDIR} >> >> +.if ${MK_LIBCPLUSPLUS} != "no" >> +LDADD+= -lcxxrt >> +DPADD+= ${LIBCXXRT} >> +.else >> +LDADD+= -lsupc++ >> +DPADD+= ${LIBSTDCPLUSPLUS} >> +.endif >> + >> SHLIB_MAJOR= 2 >> >> WITHOUT_MAN= >> >> Modified: head/lib/libproc/proc_sym.c >> ============================================================================== >> --- head/lib/libproc/proc_sym.c Thu Sep 6 02:07:58 2012 >> (r240155) >> +++ head/lib/libproc/proc_sym.c Thu Sep 6 03:19:48 2012 >> (r240156) >> @@ -46,6 +46,8 @@ >> >> #include "_libproc.h" >> >> +extern char *__cxa_demangle(const char *, char *, size_t *, int *); >> + >> static void proc_rdl2prmap(rd_loadobj_t *, prmap_t *); >> >> static void >> @@ -266,7 +268,11 @@ proc_addr2sym(struct proc_handle *p, uin >> if (addr >= rsym && addr <= (rsym + sym.st_size)) { >> s = elf_strptr(e, dynsymstridx, sym.st_name); >> if (s) { >> - strlcpy(name, s, namesz); >> + if (strlen(s) > 2 && >> + s[0] == '_' && s[1] == 'Z') > checking "strlen(s) > 2" is useless and not optimal here, you can omit it > completely > checking s[0] and s[1] is enough, it implies that length is >=2 > you may add something like "s[2] != 0" if length should be strictly >2
It's not really useless but can be considered non-optimal, yes. That said, the C++ demangler we're calling doesn't really care about being fast :-) Regards, -- Rui Paulo _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
