diff options
| author | Nat Goodspeed <nat@lindenlab.com> | 2024-07-02 13:12:40 -0400 |
|---|---|---|
| committer | Nat Goodspeed <nat@lindenlab.com> | 2024-07-02 13:12:40 -0400 |
| commit | 9a3c770a3bf430da8878a8691cee9b726a5f026c (patch) | |
| tree | 51aa40d9d0e9f9b14bd777109f36b172a1554906 /indra/newview | |
| parent | 082913dd592366cac269d1ccb7804d7dac9a0676 (diff) | |
Eliminate c_str() calls from LLControlGroup::loadFromFile() calls.
Passing std::string::c_str() to a (const std::string&) function parameter is
worse than clutter, it's pointless overhead: it forces the compiler to
construct a new std::string instance, instead of passing a const reference to
the one you already have in hand.
Diffstat (limited to 'indra/newview')
| -rw-r--r-- | indra/newview/tests/llluamanager_test.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/indra/newview/tests/llluamanager_test.cpp b/indra/newview/tests/llluamanager_test.cpp index 55e87acaea..824ddd445b 100644 --- a/indra/newview/tests/llluamanager_test.cpp +++ b/indra/newview/tests/llluamanager_test.cpp @@ -69,7 +69,7 @@ namespace tut auto settings{ newview / "app_settings" / "settings.xml" }; // true suppresses implicit declare; implicit declare requires // that every variable in settings.xml has a Comment, which many don't. - gSavedSettings.loadFromFile(settings.u8string().c_str(), true); + gSavedSettings.loadFromFile(settings.u8string(), true); // At test time, since we don't have the app bundle available, // extend LuaRequirePath to include the require directory in the // source tree. |