Skip to content

Commit 39ce914

Browse files
ferdymercurylmoureaux
authored andcommitted
[nfc] improve TFormula docu
[nfc][hist] clarify docu [nfc] typo vepadulano and lmoureaux thks Co-authored-by: Louis Moureaux <m_louis30@yahoo.com>
1 parent 75e9f56 commit 39ce914

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

hist/hist/src/TFormula.cxx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -411,9 +411,9 @@ Bool_t TFormula::IsHexadecimal(const TString & formula, int i)
411411
// }
412412
return false;
413413
}
414+
414415
////////////////////////////////////////////////////////////////////////////
415-
// check is given position is in a parameter name i.e. within "[ ]"
416-
////
416+
/// check if given position is in a parameter name i.e. within "[ ]"
417417
Bool_t TFormula::IsAParameterName(const TString & formula, int pos) {
418418

419419
Bool_t foundOpenParenthesis = false;
@@ -1833,7 +1833,7 @@ void TFormula::HandleExponentiation(TString &formula)
18331833

18341834

18351835
////////////////////////////////////////////////////////////////////////////////
1836-
/// Handle linear functions defined with the operator ++.
1836+
/// Handle linear functions defined with the operator ++ (@ is the shorthand).
18371837

18381838
void TFormula::HandleLinear(TString &formula)
18391839
{
@@ -1866,7 +1866,7 @@ void TFormula::HandleLinear(TString &formula)
18661866

18671867
////////////////////////////////////////////////////////////////////////////////
18681868
/// Preprocessing of formula
1869-
/// Replace all ** by ^, and removes spaces.
1869+
/// Replace all ** by ^, all ++ by @, and removes spaces.
18701870
/// Handle also parametrized functions like polN,gaus,expo,landau
18711871
/// and exponentiation.
18721872
/// Similar functionality should be added here.
@@ -1881,7 +1881,7 @@ void TFormula::PreProcessFormula(TString &formula)
18811881
HandleParamRanges(formula);
18821882
HandleFunctionArguments(formula);
18831883
HandleExponentiation(formula);
1884-
// "++" wil be dealt with Handle Linear
1884+
// "++" (now @ since we just called ReplaceAll a few lines above) will be dealt with HandleLinear
18851885
HandleLinear(formula);
18861886
// special case for "--" and "++"
18871887
// ("++" needs to be written with whitespace that is removed before but then we re-add it again

0 commit comments

Comments
 (0)