From: Kristis M. <kri...@as...> - 2004-11-13 20:08:27
|
The attached patch will result in opreport to display the following: 00000000 53 0.0621 sys_brk_jit (no_symbols) rather than: 00000000 53 0.0621 sys_brk_jit (no symbols) Please apply. ChangeLog: * libpp/name_storage.cpp(): Report "(no_symbols)" when no symbols exist in a module, which is a single token rather than the space separated "(no symbols)". Thanks, Kristis |
From: John L. <le...@mo...> - 2004-11-13 20:19:27
|
On Sat, Nov 13, 2004 at 01:08:15PM -0700, Kristis Makris wrote: > The attached patch will result in opreport to display the following: > > 00000000 53 0.0621 sys_brk_jit (no_symbols) > > rather than: > > 00000000 53 0.0621 sys_brk_jit (no symbols) > > Please apply. I don't like this. This output is human-readable, not machine-readable. It's a mistake to try and produce both (and C++ demangling will produce spaces anyway). If you want machine-readable output, then we should produce either CSV or XML. (The patch was missing anyway ...) regards john |