Rename local variables "m" and "regs" to less conflicting names
authorMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Mon, 28 Mar 2011 22:12:29 +0000 (18:12 -0400)
committerMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Mon, 28 Mar 2011 22:12:29 +0000 (18:12 -0400)
commit9508e4d5ffbb4ba602d15de0fa7843a004b02f02
treea3ba5522bb10d7664bcbe202492f77b5de14c029
parent841f7e9cd6909d5fc1d782d7d1d385b568033869
Rename local variables "m" and "regs" to less conflicting names

Rename "m" local macro variable to "__marker_counter_ptr".
Rename "regs" local macro variable to "__marker_regs".

Initial report:

The following trivial piece of code will result in corrupt events:

void dump_ackerman_current( int m, int n )
{
    trace_mark( ackerman, current, "m %d n %d", m, n );
}

This is because in ust/marker.h line 173 the definition of the
__trace_mark_counter macro uses a temporary named m (which shadows the
original m). It might be a good idea to use underscore variable names
inside these macros to make naming conflicts less likely.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Reported-by: Paul Wögerer <paul_woegerer@mentor.com>
include/ust/marker.h
This page took 0.024388 seconds and 4 git commands to generate.