p_psCroot in psTrace.c is static, but has a (private) external name

[rhl@…]

The code conventions state that a p_ps or ps prefix should be used if and only if a name is globally
visible; this is violated for p_psCroot (and p_psTraceFP)

static p_psComponent* p_psCroot = NULL; The root of the trace component

rel2_2

[gusciora@…]

Variable names have been changed to cRoot and traceFP.

[Paul Price]

To clarify, added to the code conventions document, as suggested by RHL:

A name (whether of a variable, a function, or a type) shall start \code{ps} (or
\code{p_ps}) if and only if it is visible at global scope. The distinction is
that \code{p_ps} names are not part of the documented APIs, but need to be
exposed for some reason.