Re: [PATCH 5/6] dyndbg: in dynamic_emit_prefix, change inter-field separator

2012-09-24 Thread Jason Baron
On Tue, Sep 18, 2012 at 07:56:42PM -0700, Joe Perches wrote: > On Tue, 2012-09-18 at 17:36 -0600, Jim Cromie wrote: > > dynamic_emit_prefix() currently separates modname, funcname, lineno > > with ':'. This is confounds use of cut -d: , since the field > > positions can change per callsite with

Re: [PATCH 5/6] dyndbg: in dynamic_emit_prefix, change inter-field separator

2012-09-24 Thread Jason Baron
On Tue, Sep 18, 2012 at 07:56:42PM -0700, Joe Perches wrote: On Tue, 2012-09-18 at 17:36 -0600, Jim Cromie wrote: dynamic_emit_prefix() currently separates modname, funcname, lineno with ':'. This is confounds use of cut -d: logfile, since the field positions can change per callsite with

Re: [PATCH 5/6] dyndbg: in dynamic_emit_prefix, change inter-field separator

2012-09-18 Thread Joe Perches
On Tue, 2012-09-18 at 17:36 -0600, Jim Cromie wrote: > dynamic_emit_prefix() currently separates modname, funcname, lineno > with ':'. This is confounds use of cut -d: , since the field > positions can change per callsite with dynamic-debug. So change > inter-field separator to '.' and keep the

[PATCH 5/6] dyndbg: in dynamic_emit_prefix, change inter-field separator

2012-09-18 Thread Jim Cromie
dynamic_emit_prefix() currently separates modname, funcname, lineno with ':'. This is confounds use of cut -d: , since the field positions can change per callsite with dynamic-debug. So change inter-field separator to '.' and keep the ':' prefix terminator. This improves the situation, but

[PATCH 5/6] dyndbg: in dynamic_emit_prefix, change inter-field separator

2012-09-18 Thread Jim Cromie
dynamic_emit_prefix() currently separates modname, funcname, lineno with ':'. This is confounds use of cut -d: logfile, since the field positions can change per callsite with dynamic-debug. So change inter-field separator to '.' and keep the ':' prefix terminator. This improves the situation,

Re: [PATCH 5/6] dyndbg: in dynamic_emit_prefix, change inter-field separator

2012-09-18 Thread Joe Perches
On Tue, 2012-09-18 at 17:36 -0600, Jim Cromie wrote: dynamic_emit_prefix() currently separates modname, funcname, lineno with ':'. This is confounds use of cut -d: logfile, since the field positions can change per callsite with dynamic-debug. So change inter-field separator to '.' and keep