diff options
author | Nat Goodspeed <nat@lindenlab.com> | 2013-07-26 08:58:47 -0400 |
---|---|---|
committer | Nat Goodspeed <nat@lindenlab.com> | 2013-07-26 08:58:47 -0400 |
commit | c109361861a9f96566ae4acbc2b42e6afd5180bf (patch) | |
tree | 092e646d038fc70a13c86bb53efc6014a5bb53e7 /indra | |
parent | d2386652f8a44c5adcc7fbcc65741649dd90aa2b (diff) |
CHOP-962: per code review, comment that other code relies on "Global".
The comment advises grepping for "Global" rather than specifically pointing to
llcontrol.cpp because that's NOT the only place that relies on the name
"Global"! Besides, by the time someone does want to change the name, still
other such dependencies could've crept in.
Diffstat (limited to 'indra')
-rwxr-xr-x | indra/newview/llviewercontrol.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/indra/newview/llviewercontrol.cpp b/indra/newview/llviewercontrol.cpp index afbb59e723..744ec4de2b 100755 --- a/indra/newview/llviewercontrol.cpp +++ b/indra/newview/llviewercontrol.cpp @@ -83,7 +83,9 @@ BOOL gHackGodmode = FALSE; #endif - +// Should you contemplate changing the name "Global", please first grep for +// that string literal. There are at least a couple other places in the C++ +// code that assume the LLControlGroup named "Global" is gSavedSettings. LLControlGroup gSavedSettings("Global"); // saved at end of session LLControlGroup gSavedPerAccountSettings("PerAccount"); // saved at end of session LLControlGroup gCrashSettings("CrashSettings"); // saved at end of session |