Replace shared_str operator* by c_str() for xrAICore and xrScriptEngine

Xottab-DUTY · Xottab-DUTY · commit 54d1508a140e · 2017-11-13T18:07:12.000+05:00
diff --git a/src/xrAICore/Navigation/PatrolPath/patrol_path_params.cpp b/src/xrAICore/Navigation/PatrolPath/patrol_path_params.cpp
@@ -41,7 +41,7 @@ const Fvector& CPatrolPathParams::point(u32 index) const
     if (!m_path->vertex(index))
     {
         GEnv.ScriptEngine->script_log(LuaMessageType::Error,
-            "Can't get information about patrol point number %d in the patrol way %s", index, *m_path_name);
+            "Can't get information about patrol point number %d in the patrol way %s", index, m_path_name.c_str());
         index = (*m_path->vertices().begin()).second->vertex_id();
     }
     VERIFY(m_path->vertex(index));
@@ -83,7 +83,7 @@ Flags32 CPatrolPathParams::flags(u32 index) const
 LPCSTR CPatrolPathParams::name(u32 index) const
 {
     VERIFY(m_path->vertex(index));
-    return (*m_path->vertex(index)->data().name());
+    return (m_path->vertex(index)->data().name().c_str());
 }
 
 bool CPatrolPathParams::terminal(u32 index) const
diff --git a/src/xrAICore/Navigation/PatrolPath/patrol_path_storage.cpp b/src/xrAICore/Navigation/PatrolPath/patrol_path_storage.cpp
@@ -32,7 +32,7 @@ void CPatrolPathStorage::load_raw(
         shared_str patrol_name;
         sub_chunk->r_stringZ(patrol_name);
         const_iterator I = m_registry.find(patrol_name);
-        VERIFY3(I == m_registry.end(), "Duplicated patrol path found", *patrol_name);
+        VERIFY3(I == m_registry.end(), "Duplicated patrol path found", patrol_name.c_str());
         m_registry.insert(std::make_pair(
             patrol_name, &(new CPatrolPath(patrol_name))->load_raw(level_graph, cross, game_graph, *sub_chunk)));
     }
@@ -70,7 +70,7 @@ void CPatrolPathStorage::load(IReader& stream)
         chunk1->close();
 
         const_iterator I = m_registry.find(pair.first);
-        VERIFY3(I == m_registry.end(), "Duplicated patrol path found ", *pair.first);
+        VERIFY3(I == m_registry.end(), "Duplicated patrol path found ", pair.first.c_str());
 
 #ifdef DEBUG
         pair.second->name(pair.first);
diff --git a/src/xrAICore/Navigation/PatrolPath/patrol_path_storage_inline.h b/src/xrAICore/Navigation/PatrolPath/patrol_path_storage_inline.h
@@ -15,7 +15,7 @@ IC const CPatrolPath* CPatrolPathStorage::path(shared_str patrol_name, bool no_a
     const_iterator I = patrol_paths().find(patrol_name);
     if (I == patrol_paths().end())
     {
-        THROW3(no_assert, "There is no patrol path", *patrol_name);
+        THROW3(no_assert, "There is no patrol path", patrol_name.c_str());
         return (0);
     }
     return ((*I).second);
diff --git a/src/xrAICore/Navigation/PatrolPath/patrol_point.cpp b/src/xrAICore/Navigation/PatrolPath/patrol_point.cpp
@@ -41,7 +41,7 @@ void CPatrolPoint::verify_vertex_id(
 
     VERIFY(m_path);
     string1024 temp;
-    xr_sprintf(temp, "\n! Patrol point %s in path %s is not on the level graph vertex!", *m_name, *m_path->m_name);
+    xr_sprintf(temp, "\n! Patrol point %s in path %s is not on the level graph vertex!", m_name.c_str(), m_path->m_name.c_str());
     THROW2(level_graph->valid_vertex_id(m_level_vertex_id), temp);
 }
 #endif
diff --git a/src/xrScriptEngine/script_debugger.cpp b/src/xrScriptEngine/script_debugger.cpp
@@ -378,7 +378,7 @@ bool CScriptDebugger::HasBreakPoint(const char* fileName, s32 lineNum)
     for (u32 i = 0; i < m_breakPoints.size(); i++)
     {
         SBreakPoint bp(m_breakPoints[i]);
-        if (bp.nLine == lineNum && xr_strlen(bp.fileName) == xr_strlen(sFileName) && !_stricmp(*bp.fileName, sFileName))
+        if (bp.nLine == lineNum && xr_strlen(bp.fileName) == xr_strlen(sFileName) && !_stricmp(bp.fileName.c_str(), sFileName))
             return true;
     }
     return false;
diff --git a/src/xrScriptEngine/script_debugger_threads.cpp b/src/xrScriptEngine/script_debugger_threads.cpp
@@ -30,8 +30,8 @@ u32 CDbgScriptThreads::FillFrom(CScriptProcess* sp)
         th.lua = scriptThread->lua();
         th.scriptID = scriptThread->thread_reference();
         th.active = scriptThread->active();
-        xr_strcat(th.name, *scriptThread->script_name());
-        xr_strcat(th.process, *sp->name());
+        xr_strcat(th.name, scriptThread->script_name().c_str());
+        xr_strcat(th.process, sp->name().c_str());
         m_threads.push_back(th);
     }
     return m_threads.size();
diff --git a/src/xrScriptEngine/script_engine.cpp b/src/xrScriptEngine/script_engine.cpp
@@ -1258,7 +1258,7 @@ CScriptThread* CScriptEngine::CreateScriptThread(LPCSTR caNamespaceName, bool do
 void CScriptEngine::DestroyScriptThread(const CScriptThread* thread)
 {
 #ifdef DEBUG
-    Msg("* Destroying script thread %s", *thread->script_name());
+    Msg("* Destroying script thread %s", thread->script_name().c_str());
 #endif
     try
     {
diff --git a/src/xrScriptEngine/script_process.cpp b/src/xrScriptEngine/script_process.cpp
@@ -19,11 +19,11 @@ CScriptProcess::CScriptProcess(CScriptEngine* scriptEngine, shared_str name, sha
     this->scriptEngine = scriptEngine;
     m_name = name;
 #ifdef DEBUG
-    Msg("* Initializing %s script process", *m_name);
+    Msg("* Initializing %s script process", m_name.c_str());
 #endif
     string256 I;
-    for (u32 i = 0, n = _GetItemCount(*scripts); i < n; i++)
-        add_script(_GetItem(*scripts, i, I), false, false);
+    for (u32 i = 0, n = _GetItemCount(scripts.c_str()); i < n; i++)
+        add_script(_GetItem(scripts.c_str(), i, I), false, false);
     m_iterator = 0;
 }
 
diff --git a/src/xrScriptEngine/script_thread.cpp b/src/xrScriptEngine/script_thread.cpp
@@ -56,14 +56,14 @@ CScriptThread::CScriptThread(CScriptEngine* scriptEngine, LPCSTR caNamespaceName
                 l_iErrorCode = lua_pcall(engineLua, 0, 0, 0);
                 if (l_iErrorCode)
                 {
-                    CScriptEngine::print_output(engineLua, *m_script_name, l_iErrorCode);
+                    CScriptEngine::print_output(engineLua, m_script_name.c_str(), l_iErrorCode);
                     CScriptEngine::on_error(engineLua);
                     return;
                 }
             }
             else
             {
-                CScriptEngine::print_output(engineLua, *m_script_name, l_iErrorCode);
+                CScriptEngine::print_output(engineLua, m_script_name.c_str(), l_iErrorCode);
                 CScriptEngine::on_error(engineLua);
                 return;
             }
@@ -107,7 +107,7 @@ bool CScriptThread::update()
         int l_iErrorCode = lua_resume(lua(), 0);
         if (l_iErrorCode && l_iErrorCode != LUA_YIELD)
         {
-            CScriptEngine::print_output(lua(), *script_name(), l_iErrorCode);
+            CScriptEngine::print_output(lua(), script_name().c_str(), l_iErrorCode);
             CScriptEngine::on_error(scriptEngine->lua());
 #ifdef DEBUG
             print_stack(lua());
@@ -121,14 +121,14 @@ bool CScriptThread::update()
 #ifdef DEBUG
                 if (m_current_stack_level)
                 {
-                    CScriptEngine::print_output(lua(), *script_name(), l_iErrorCode);
+                    CScriptEngine::print_output(lua(), script_name().c_str(), l_iErrorCode);
                     CScriptEngine::on_error(scriptEngine->lua());
                     // print_stack(lua());
                 }
 #endif
                 m_active = false;
 #ifdef DEBUG
-                scriptEngine->script_log(LuaMessageType::Info, "Script %s is finished!", *m_script_name);
+                scriptEngine->script_log(LuaMessageType::Info, "Script %s is finished!", m_script_name.c_str());
 #endif
             }
             else

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ const Fvector& CPatrolPathParams::point(u32 index) const`
`41`	`41`	`if (!m_path->vertex(index))`
`42`	`42`	`{`
`43`	`43`	`GEnv.ScriptEngine->script_log(LuaMessageType::Error,`
`44`		`- "Can't get information about patrol point number %d in the patrol way %s", index, *m_path_name);`
	`44`	`+ "Can't get information about patrol point number %d in the patrol way %s", index, m_path_name.c_str());`
`45`	`45`	`index = (*m_path->vertices().begin()).second->vertex_id();`
`46`	`46`	`}`
`47`	`47`	`VERIFY(m_path->vertex(index));`
`@@ -83,7 +83,7 @@ Flags32 CPatrolPathParams::flags(u32 index) const`
`83`	`83`	`LPCSTR CPatrolPathParams::name(u32 index) const`
`84`	`84`	`{`
`85`	`85`	`VERIFY(m_path->vertex(index));`
`86`		`- return (*m_path->vertex(index)->data().name());`
	`86`	`+ return (m_path->vertex(index)->data().name().c_str());`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`bool CPatrolPathParams::terminal(u32 index) const`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ IC const CPatrolPath* CPatrolPathStorage::path(shared_str patrol_name, bool no_a`
`15`	`15`	`const_iterator I = patrol_paths().find(patrol_name);`
`16`	`16`	`if (I == patrol_paths().end())`
`17`	`17`	`{`
`18`		`- THROW3(no_assert, "There is no patrol path", *patrol_name);`
	`18`	`+ THROW3(no_assert, "There is no patrol path", patrol_name.c_str());`
`19`	`19`	`return (0);`
`20`	`20`	`}`
`21`	`21`	`return ((*I).second);`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ void CPatrolPoint::verify_vertex_id(`
`41`	`41`
`42`	`42`	`VERIFY(m_path);`
`43`	`43`	`string1024 temp;`
`44`		`- xr_sprintf(temp, "\n! Patrol point %s in path %s is not on the level graph vertex!", m_name, m_path->m_name);`
	`44`	`+ xr_sprintf(temp, "\n! Patrol point %s in path %s is not on the level graph vertex!", m_name.c_str(), m_path->m_name.c_str());`
`45`	`45`	`THROW2(level_graph->valid_vertex_id(m_level_vertex_id), temp);`
`46`	`46`	`}`
`47`	`47`	`#endif`
Original file line number	Diff line number	Diff line change
`@@ -378,7 +378,7 @@ bool CScriptDebugger::HasBreakPoint(const char* fileName, s32 lineNum)`
`378`	`378`	`for (u32 i = 0; i < m_breakPoints.size(); i++)`
`379`	`379`	`{`
`380`	`380`	`SBreakPoint bp(m_breakPoints[i]);`
`381`		`- if (bp.nLine == lineNum && xr_strlen(bp.fileName) == xr_strlen(sFileName) && !_stricmp(*bp.fileName, sFileName))`
	`381`	`+ if (bp.nLine == lineNum && xr_strlen(bp.fileName) == xr_strlen(sFileName) && !_stricmp(bp.fileName.c_str(), sFileName))`
`382`	`382`	`return true;`
`383`	`383`	`}`
`384`	`384`	`return false;`
Original file line number	Diff line number	Diff line change
`@@ -1258,7 +1258,7 @@ CScriptThread* CScriptEngine::CreateScriptThread(LPCSTR caNamespaceName, bool do`
`1258`	`1258`	`void CScriptEngine::DestroyScriptThread(const CScriptThread* thread)`
`1259`	`1259`	`{`
`1260`	`1260`	`#ifdef DEBUG`
`1261`		`- Msg("* Destroying script thread %s", *thread->script_name());`
	`1261`	`+ Msg("* Destroying script thread %s", thread->script_name().c_str());`
`1262`	`1262`	`#endif`
`1263`	`1263`	`try`
`1264`	`1264`	`{`
Original file line number	Diff line number	Diff line change
`@@ -56,14 +56,14 @@ CScriptThread::CScriptThread(CScriptEngine* scriptEngine, LPCSTR caNamespaceName`
`56`	`56`	`l_iErrorCode = lua_pcall(engineLua, 0, 0, 0);`
`57`	`57`	`if (l_iErrorCode)`
`58`	`58`	`{`
`59`		`- CScriptEngine::print_output(engineLua, *m_script_name, l_iErrorCode);`
	`59`	`+ CScriptEngine::print_output(engineLua, m_script_name.c_str(), l_iErrorCode);`
`60`	`60`	`CScriptEngine::on_error(engineLua);`
`61`	`61`	`return;`
`62`	`62`	`}`
`63`	`63`	`}`
`64`	`64`	`else`
`65`	`65`	`{`
`66`		`- CScriptEngine::print_output(engineLua, *m_script_name, l_iErrorCode);`
	`66`	`+ CScriptEngine::print_output(engineLua, m_script_name.c_str(), l_iErrorCode);`
`67`	`67`	`CScriptEngine::on_error(engineLua);`
`68`	`68`	`return;`
`69`	`69`	`}`
`@@ -107,7 +107,7 @@ bool CScriptThread::update()`
`107`	`107`	`int l_iErrorCode = lua_resume(lua(), 0);`
`108`	`108`	`if (l_iErrorCode && l_iErrorCode != LUA_YIELD)`
`109`	`109`	`{`
`110`		`- CScriptEngine::print_output(lua(), *script_name(), l_iErrorCode);`
	`110`	`+ CScriptEngine::print_output(lua(), script_name().c_str(), l_iErrorCode);`
`111`	`111`	`CScriptEngine::on_error(scriptEngine->lua());`
`112`	`112`	`#ifdef DEBUG`
`113`	`113`	`print_stack(lua());`
`@@ -121,14 +121,14 @@ bool CScriptThread::update()`
`121`	`121`	`#ifdef DEBUG`
`122`	`122`	`if (m_current_stack_level)`
`123`	`123`	`{`
`124`		`- CScriptEngine::print_output(lua(), *script_name(), l_iErrorCode);`
	`124`	`+ CScriptEngine::print_output(lua(), script_name().c_str(), l_iErrorCode);`
`125`	`125`	`CScriptEngine::on_error(scriptEngine->lua());`
`126`	`126`	`// print_stack(lua());`
`127`	`127`	`}`
`128`	`128`	`#endif`
`129`	`129`	`m_active = false;`
`130`	`130`	`#ifdef DEBUG`
`131`		`- scriptEngine->script_log(LuaMessageType::Info, "Script %s is finished!", *m_script_name);`
	`131`	`+ scriptEngine->script_log(LuaMessageType::Info, "Script %s is finished!", m_script_name.c_str());`
`132`	`132`	`#endif`
`133`	`133`	`}`
`134`	`134`	`else`