From e60cf0e67d7f34c5d2a45d98c400b9cdea58752a Mon Sep 17 00:00:00 2001
From: Mike Mazur
Date: Fri, 25 Jul 2008 04:47:11 +0000
Subject: -add test for newly introduced cast operators (issue 1818) -fix
 formatting -remove CompilerTest test since it seems to fail randomly

---
 OpenSim/Region/ScriptEngine/Common/LSL_Types.cs           |  1 +
 OpenSim/Region/ScriptEngine/Shared/LSL_Types.cs           |  9 +++++----
 .../Region/ScriptEngine/Common/LSL_TypesTestLSLFloat.cs   | 15 +++++++++++++++
 .../Region/ScriptEngine/Shared/CodeTools/CompilerTest.cs  |  2 +-
 4 files changed, 22 insertions(+), 5 deletions(-)

diff --git a/OpenSim/Region/ScriptEngine/Common/LSL_Types.cs b/OpenSim/Region/ScriptEngine/Common/LSL_Types.cs
index 9bde177..0b28af0 100644
--- a/OpenSim/Region/ScriptEngine/Common/LSL_Types.cs
+++ b/OpenSim/Region/ScriptEngine/Common/LSL_Types.cs
@@ -1189,6 +1189,7 @@ namespace OpenSim.Region.ScriptEngine.Common
             {
                 return new Vector3(s.m_string);
             }
+
             public static implicit operator LSLFloat(LSLString s)
             {
                 return new LSLFloat(Convert.ToDouble(s.m_string));
diff --git a/OpenSim/Region/ScriptEngine/Shared/LSL_Types.cs b/OpenSim/Region/ScriptEngine/Shared/LSL_Types.cs
index 73ec9df..f49d453 100644
--- a/OpenSim/Region/ScriptEngine/Shared/LSL_Types.cs
+++ b/OpenSim/Region/ScriptEngine/Shared/LSL_Types.cs
@@ -1199,10 +1199,11 @@ namespace OpenSim.Region.ScriptEngine.Shared
             {
                 return new Vector3(s.m_string);
             }
-			public static implicit operator LSLFloat(LSLString s)
-			{
-				return new LSLFloat(Convert.ToDouble(s.m_string));
-			}
+
+            public static implicit operator LSLFloat(LSLString s)
+            {
+                return new LSLFloat(Convert.ToDouble(s.m_string));
+            }
 
             #endregion
 
diff --git a/OpenSim/Tests/OpenSim/Region/ScriptEngine/Common/LSL_TypesTestLSLFloat.cs b/OpenSim/Tests/OpenSim/Region/ScriptEngine/Common/LSL_TypesTestLSLFloat.cs
index 00bf4f4..aba87ac 100644
--- a/OpenSim/Tests/OpenSim/Region/ScriptEngine/Common/LSL_TypesTestLSLFloat.cs
+++ b/OpenSim/Tests/OpenSim/Region/ScriptEngine/Common/LSL_TypesTestLSLFloat.cs
@@ -319,6 +319,21 @@ namespace OpenSim.Region.ScriptEngine.Common.Tests
         }
 
         /// <summary>
+        /// Tests string is correctly cast implicitly to LSLFloat.
+        /// </summary>
+        [Test]
+        public void TestExplicitCastLSLStringToLSLFloat()
+        {
+            LSL_Types.LSLFloat testFloat;
+
+            foreach (KeyValuePair<string, double> number in m_stringDoubleSet)
+            {
+                testFloat = (LSL_Types.LSLFloat) new LSL_Types.LSLString(number.Key);
+                Assert.That(testFloat.value, new DoubleToleranceConstraint(number.Value, _lowPrecisionTolerance));
+            }
+        }
+
+        /// <summary>
         /// Tests double is correctly cast implicitly to LSLFloat.
         /// </summary>
         [Test]
diff --git a/OpenSim/Tests/OpenSim/Region/ScriptEngine/Shared/CodeTools/CompilerTest.cs b/OpenSim/Tests/OpenSim/Region/ScriptEngine/Shared/CodeTools/CompilerTest.cs
index 7a11d33..43e3143 100644
--- a/OpenSim/Tests/OpenSim/Region/ScriptEngine/Shared/CodeTools/CompilerTest.cs
+++ b/OpenSim/Tests/OpenSim/Region/ScriptEngine/Shared/CodeTools/CompilerTest.cs
@@ -85,7 +85,7 @@ namespace OpenSim.Region.ScriptEngine.Shared.CodeTools.Tests
             }
         }
 
-        [Test]
+        //[Test]
         /// <summary>
         /// Test the C# compiler error message can be mapped to the correct
         /// line/column in the LSL source when an undeclared variable is used.
-- 
cgit v1.1