From 109aa00150aea01ba32620b4d965edd459827ff7 Mon Sep 17 00:00:00 2001
From: Dr Scofield
Date: Tue, 2 Sep 2008 12:07:23 +0000
Subject: fix: PostInitialise() not being called on script engines (nasty one
that) cleanup: warnings, readability
---
.../ScriptEngine/Shared/Api/Runtime/Executor.cs | 2 +-
OpenSim/Region/ScriptEngine/XEngine/XEngine.cs | 93 +++++++++++-----------
2 files changed, 46 insertions(+), 49 deletions(-)
(limited to 'OpenSim/Region/ScriptEngine')
diff --git a/OpenSim/Region/ScriptEngine/Shared/Api/Runtime/Executor.cs b/OpenSim/Region/ScriptEngine/Shared/Api/Runtime/Executor.cs
index 0241da5..aee03ef 100644
--- a/OpenSim/Region/ScriptEngine/Shared/Api/Runtime/Executor.cs
+++ b/OpenSim/Region/ScriptEngine/Shared/Api/Runtime/Executor.cs
@@ -90,7 +90,7 @@ namespace OpenSim.Region.ScriptEngine.Shared.ScriptBase
/// Make sure our object does not timeout when in AppDomain. (Called by ILease base class)
///
///
- public Object InitializeLifetimeService()
+ public override Object InitializeLifetimeService()
{
//Console.WriteLine("Executor: InitializeLifetimeService()");
// return null;
diff --git a/OpenSim/Region/ScriptEngine/XEngine/XEngine.cs b/OpenSim/Region/ScriptEngine/XEngine/XEngine.cs
index f9f8883..8ad8110 100644
--- a/OpenSim/Region/ScriptEngine/XEngine/XEngine.cs
+++ b/OpenSim/Region/ScriptEngine/XEngine/XEngine.cs
@@ -145,11 +145,11 @@ namespace OpenSim.Region.ScriptEngine.XEngine
public void Initialise(Scene scene, IConfigSource configSource)
{
AppDomain.CurrentDomain.AssemblyResolve +=
- OnAssemblyResolve;
+ OnAssemblyResolve;
m_log.InfoFormat("[XEngine] Initializing scripts in region {0}",
- scene.RegionInfo.RegionName);
- m_Scene=scene;
+ scene.RegionInfo.RegionName);
+ m_Scene = scene;
m_ScriptConfig = configSource.Configs["XEngine"];
@@ -165,34 +165,31 @@ namespace OpenSim.Region.ScriptEngine.XEngine
string priority = m_ScriptConfig.GetString("Priority", "BelowNormal");
int maxScriptQueue = m_ScriptConfig.GetInt("MaxScriptEventQueue",300);
int stackSize = m_ScriptConfig.GetInt("ThreadStackSize", 262144);
- int sleepTime = m_ScriptConfig.GetInt("MaintenanceInterval",
- 10)*1000;
+ int sleepTime = m_ScriptConfig.GetInt("MaintenanceInterval", 10)*1000;
m_EventLimit = m_ScriptConfig.GetInt("EventLimit", 30);
- m_KillTimedOutScripts = m_ScriptConfig.GetBoolean(
- "KillTimedOutScripts", false);
+ m_KillTimedOutScripts = m_ScriptConfig.GetBoolean("KillTimedOutScripts", false);
int saveTime = m_ScriptConfig.GetInt("SaveInterval", 300)*1000;
ThreadPriority prio = ThreadPriority.BelowNormal;
switch (priority)
{
case "Lowest":
- prio=ThreadPriority.Lowest;
+ prio = ThreadPriority.Lowest;
break;
case "BelowNormal":
- prio=ThreadPriority.BelowNormal;
+ prio = ThreadPriority.BelowNormal;
break;
case "Normal":
- prio=ThreadPriority.Normal;
+ prio = ThreadPriority.Normal;
break;
case "AboveNormal":
- prio=ThreadPriority.AboveNormal;
+ prio = ThreadPriority.AboveNormal;
break;
case "Highest":
- prio=ThreadPriority.Highest;
+ prio = ThreadPriority.Highest;
break;
default:
- m_log.ErrorFormat("[XEngine] Invalid thread priority: '"+
- priority+"'. Assuming BelowNormal");
+ m_log.ErrorFormat("[XEngine] Invalid thread priority: '{0}'. Assuming BelowNormal", priority);
break;
}
@@ -204,7 +201,7 @@ namespace OpenSim.Region.ScriptEngine.XEngine
m_EventManager = new EventManager(this);
StartEngine(minThreads, maxThreads, idleTimeout, prio,
- maxScriptQueue, stackSize);
+ maxScriptQueue, stackSize);
m_Compiler = new Compiler(this);
@@ -218,15 +215,14 @@ namespace OpenSim.Region.ScriptEngine.XEngine
if (sleepTime > 0)
{
- m_ThreadPool.QueueWorkItem(new WorkItemCallback(
- this.DoMaintenance), new Object[]
- { sleepTime });
+ m_ThreadPool.QueueWorkItem(new WorkItemCallback(this.DoMaintenance),
+ new Object[]{ sleepTime });
}
if (saveTime > 0)
{
- m_ThreadPool.QueueWorkItem(new WorkItemCallback(
- this.DoBackup), new Object[] { saveTime });
+ m_ThreadPool.QueueWorkItem(new WorkItemCallback(this.DoBackup),
+ new Object[] { saveTime });
}
scene.RegisterModuleInterface(this);
@@ -279,8 +275,8 @@ namespace OpenSim.Region.ScriptEngine.XEngine
instances.Clear();
- m_ThreadPool.QueueWorkItem(new WorkItemCallback(
- this.DoBackup), new Object[] { saveTime });
+ m_ThreadPool.QueueWorkItem(new WorkItemCallback(this.DoBackup),
+ new Object[] { saveTime });
return 0;
}
@@ -302,9 +298,8 @@ namespace OpenSim.Region.ScriptEngine.XEngine
System.Threading.Thread.Sleep(sleepTime);
- m_ThreadPool.QueueWorkItem(new WorkItemCallback(
- this.DoMaintenance), new Object[]
- { sleepTime });
+ m_ThreadPool.QueueWorkItem(new WorkItemCallback(this.DoMaintenance),
+ new Object[]{ sleepTime });
return 0;
}
@@ -321,25 +316,26 @@ namespace OpenSim.Region.ScriptEngine.XEngine
public void OnRezScript(uint localID, LLUUID itemID, string script, int startParam, bool postOnRez)
{
- Object[] parms = new Object[]
- { localID, itemID, script, startParam, postOnRez};
+ Object[] parms = new Object[]{localID, itemID, script, startParam, postOnRez};
lock (m_CompileQueue)
{
m_CompileQueue.Enqueue(parms);
+
if (m_CurrentCompile == null)
{
if (m_firstStart)
{
m_firstStart = false;
m_CurrentCompile = m_ThreadPool.QueueWorkItem(
- new WorkItemCallback(
- this.DoScriptWait), new Object[0]);
+ new WorkItemCallback(this.DoScriptWait),
+ new Object[0]);
return;
}
+
m_CurrentCompile = m_ThreadPool.QueueWorkItem(
- new WorkItemCallback(
- this.DoOnRezScriptQueue), new Object[0]);
+ new WorkItemCallback(this.DoOnRezScriptQueue),
+ new Object[0]);
}
}
}
@@ -353,8 +349,8 @@ namespace OpenSim.Region.ScriptEngine.XEngine
if (m_CompileQueue.Count > 0)
{
m_CurrentCompile = m_ThreadPool.QueueWorkItem(
- new WorkItemCallback(
- this.DoOnRezScriptQueue), new Object[0]);
+ new WorkItemCallback(this.DoOnRezScriptQueue),
+ new Object[0]);
}
else
{
@@ -384,8 +380,8 @@ namespace OpenSim.Region.ScriptEngine.XEngine
if (m_CompileQueue.Count > 0)
{
m_CurrentCompile = m_ThreadPool.QueueWorkItem(
- new WorkItemCallback(
- this.DoOnRezScriptQueue), new Object[0]);
+ new WorkItemCallback(this.DoOnRezScriptQueue),
+ new Object[0]);
}
else
{
@@ -459,7 +455,7 @@ namespace OpenSim.Region.ScriptEngine.XEngine
if ((!m_Scripts.ContainsKey(itemID)) ||
(m_Scripts[itemID].AssetID != assetID))
{
- LLUUID appDomain=assetID;
+ LLUUID appDomain = assetID;
if (part.ParentGroup.RootPart.IsAttachment)
appDomain = part.ParentGroup.RootPart.UUID;
@@ -473,8 +469,7 @@ namespace OpenSim.Region.ScriptEngine.XEngine
// "ScriptEngines",
// m_Scene.RegionInfo.RegionID.ToString());
- Evidence baseEvidence =
- AppDomain.CurrentDomain.Evidence;
+ Evidence baseEvidence = AppDomain.CurrentDomain.Evidence;
Evidence evidence = new Evidence(baseEvidence);
m_AppDomains[appDomain] =
@@ -489,18 +484,19 @@ namespace OpenSim.Region.ScriptEngine.XEngine
}
catch (Exception e)
{
- m_log.Error("[XEngine] Exception creating app domain:\n"+e.ToString());
+ m_log.ErrorFormat("[XEngine] Exception creating app domain:\n {0}", e.ToString());
return false;
}
}
m_DomainScripts[appDomain].Add(itemID);
- ScriptInstance instance = new ScriptInstance(this,localID,
- part.UUID, itemID, assetID, assembly,
- m_AppDomains[appDomain],
- part.ParentGroup.RootPart.Name,
- item.Name, startParam, postOnRez,
- StateSource.NewRez, m_MaxScriptQueue);
+ ScriptInstance instance =
+ new ScriptInstance(this,localID,
+ part.UUID, itemID, assetID, assembly,
+ m_AppDomains[appDomain],
+ part.ParentGroup.RootPart.Name,
+ item.Name, startParam, postOnRez,
+ StateSource.NewRez, m_MaxScriptQueue);
m_log.DebugFormat("[XEngine] Loaded script {0}.{1}",
part.ParentGroup.RootPart.Name, item.Name);
@@ -657,8 +653,9 @@ namespace OpenSim.Region.ScriptEngine.XEngine
//
public IScriptWorkItem QueueEventHandler(object parms)
{
- return new XWorkItem(m_ThreadPool.QueueWorkItem(new WorkItemCallback(
- this.ProcessEventHandler), parms));
+ return new XWorkItem(m_ThreadPool.QueueWorkItem(
+ new WorkItemCallback(this.ProcessEventHandler),
+ parms));
}
//
@@ -723,7 +720,7 @@ namespace OpenSim.Region.ScriptEngine.XEngine
string[] pathList = new string[] {"bin", "ScriptEngines",
Path.Combine("ScriptEngines",
m_Scene.RegionInfo.RegionID.ToString())};
-
+
string assemblyName = args.Name;
if (assemblyName.IndexOf(",") != -1)
assemblyName = args.Name.Substring(0, args.Name.IndexOf(","));
--
cgit v1.1