Patch from Melanie: 0001077: [PATCH] LSL types cannot be cast implicitly or explicitly in many cases Thanks Melanie!

* Also, I moved the event parser and re-writer to a separate static object.   More work will be done here shortly.

1 files changed, 9 insertions, 241 deletions

diff --git a/OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/LSL2CSConverter.cs b/OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/LSL2CSConverter.cs
index f1e6ebf..06be20e 100644
--- a/OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/LSL2CSConverter.cs
+++ b/OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/LSL2CSConverter.cs
@@ -42,16 +42,16 @@ namespace OpenSim.Region.ScriptEngine.DotNetEngine.Compiler.LSL
         //private Regex rnw = new Regex(@"[a-zA-Z0-9_\-]", RegexOptions.Compiled);
         private Dictionary<string, string> dataTypes = new Dictionary<string, string>();
         private Dictionary<string, string> quotes = new Dictionary<string, string>();
-        private Dictionary<string, scriptEvents> state_events = new Dictionary<string, scriptEvents>();
+        
 
         public LSL2CSConverter()
         {
             // Only the types we need to convert
             dataTypes.Add("void", "void");
-            dataTypes.Add("integer", "int");
+            dataTypes.Add("integer", "LSL_Types.LSLInteger");
             dataTypes.Add("float", "double");
-            dataTypes.Add("string", "string");
-            dataTypes.Add("key", "string");
+            dataTypes.Add("string", "LSL_Types.LSLString");
+            dataTypes.Add("key", "LSL_Types.LSLString");
             dataTypes.Add("vector", "LSL_Types.Vector3");
             dataTypes.Add("rotation", "LSL_Types.Quaternion");
             dataTypes.Add("list", "LSL_Types.list");
@@ -314,100 +314,7 @@ namespace OpenSim.Region.ScriptEngine.DotNetEngine.Compiler.LSL
             }
 
 
-            // Finds out which events are in the script and writes a method call with the events in each state_entry event
-
-            // Note the (?:)? block optional, and not returning a group.   Less greedy then .*
-
-            string[] eventmatches = new string[0];
-            //Regex stateevents = new Regex(@"(public void )([^_]+)(_event_)([^\(]+)[\(\)]+\s+[^\{]\{");
-            eventmatches = Regex.Split(Script, @"public void\s([^_]+)_event_([^\(]+)\((?:[a-zA-Z0-9\s_,\.\-]+)?\)(?:[^\{]+)?\{", RegexOptions.Compiled | RegexOptions.Multiline | RegexOptions.Singleline | RegexOptions.IgnoreCase);
-            for (int pos = 0; pos<eventmatches.GetUpperBound(0);pos++)
-            {
-                pos++; // garbage
-
-                string statea = eventmatches[pos]; pos++;
-                string eventa = eventmatches[pos];
-                scriptEvents storedEventsForState = scriptEvents.None;
-                if (state_events.ContainsKey(statea))
-                {
-                    storedEventsForState = state_events[statea];
-                    state_events[statea] |= convertnametoFlag(eventa);
-                }
-                else
-                {
-                    state_events.Add(statea, convertnametoFlag(eventa));
-                }
-                Console.WriteLine("State:" + statea + ", event: " + eventa);
-            }
-            Console.WriteLine("Matches:" + eventmatches.GetUpperBound(0));
-            // Add namespace, class name and inheritance
-
-            // Looking *ONLY* for state entry events
-            string scriptCopy = "";
-            
-            //Only match State_Entry events now
-            // Note the whole regex is a group, then we have the state this entry belongs to.
-            eventmatches = Regex.Split(Script, @"(public void\s([^_]+)_event_state_entry[\(\)](?:[^\{]+)?\{)", RegexOptions.Compiled | RegexOptions.Multiline | RegexOptions.Singleline | RegexOptions.IgnoreCase);
-            int endloop = eventmatches.GetUpperBound(0);
-
-            // Add all the states to a list of 
-            List<string> unUsedStates = new List<string>();
-            foreach (string state in state_events.Keys)
-            {
-                unUsedStates.Add(state);
-            }
-
-            // If endloop is 0, then there are no state entry events in the entire script.
-            // Stick a default state entry in there.
-            if (endloop == 0)
-            {
-                if (state_events.ContainsKey("default"))
-                {
-                    scriptCopy = "\r\n// programmatically added this state entry event.\r\n\r\npublic void default_event_state_entry() {\r\n\tosSetStateEvents((int)" + (int)state_events["default"] + ");\r\n }\r\n\r\n " + Script;
-                    unUsedStates.Remove("default");
-                }
-                else 
-                {
-                    throw new Exception("You must define a default state. Compile failed.  See LSL documentation for more details.");
-                }
-            }
-
-            // Loop over state entry events and rewrite the first line to define the events the state listens for.
-            for (int pos = 0; pos < endloop; pos++)
-            {
-                // Returns text before state entry match,
-                scriptCopy += eventmatches[pos]; pos++;
-
-                // Returns text of state entry match, 
-                scriptCopy += eventmatches[pos]; pos++;
-                 
-                // Returns which state we're matching and writes a method call to the end of the above state_entry
-                scriptCopy += "\r\n\t\tosSetStateEvents((int)" + (int)state_events[eventmatches[pos]] + ");"; //pos++;
-                
-                // Remove the state from the unused list.   There might be more states matched then defined, so we 
-                // check if the state was defined first
-                if (unUsedStates.Contains(eventmatches[pos]))
-                    unUsedStates.Remove(eventmatches[pos]);
-                
-                // adds the remainder of the script.
-                if ((pos + 1) == endloop)
-                {
-                    pos++;
-                    scriptCopy += eventmatches[pos++];
-                }
-                
-            }
-
-            // states with missing state_entry blocks won't publish their events, 
-            // so, to fix that we write a state entry with only the event publishing method for states missing a state_entry event
-            foreach (string state in unUsedStates)
-            {
-                // Write the remainder states out into a blank state entry with the event setting routine
-                scriptCopy = "\r\n// programmatically added this state entry event.\r\n\r\npublic void " + state + "_event_state_entry() {\r\n\tosSetStateEvents((int)" + (int)state_events[state] + ");\r\n}\r\n\r\n " + scriptCopy;
-            }
-
-            // save modified script.
-            Script = scriptCopy;
+           
             //System.Console.WriteLine(Script);
             Return = String.Empty;// +
                      //"using OpenSim.Region.ScriptEngine.Common; using System.Collections.Generic;";
@@ -421,152 +328,13 @@ namespace OpenSim.Region.ScriptEngine.DotNetEngine.Compiler.LSL
             //Return += @"public Script() { } ";
             Return += Script;
             //Return += "} }\r\n";
-            unUsedStates.Clear();
-            state_events.Clear();
+            
             quotes.Clear();
              
             return Return;
         }
-        public scriptEvents convertnametoFlag(string eventname)
-        {
-            switch (eventname)
-            {
-                case "attach":
-                    return scriptEvents.attach;
-                    //break;
-               // case "at_rot_target":
-                    //return (long)scriptEvents.at_rot_target;
-                    //break;
-                case "at_target":
-                    return scriptEvents.at_target;
-                    //break;
-                //case "changed":
-                    //return (long)scriptEvents.changed;
-                    //break;
-                case "collision":
-                    return scriptEvents.collision;
-                   // break;
-                case "collision_end":
-                    return scriptEvents.collision_end;
-                    //break;
-                case "collision_start":
-                    return scriptEvents.collision_start;
-                   // break;
-                case "control":
-                    return scriptEvents.control;
-                    //break;
-                case "dataserver":
-                    return scriptEvents.dataserver;
-                   // break;
-                case "email":
-                    return scriptEvents.email;
-                   // break;
-                case "http_response":
-                    return scriptEvents.http_response;
-                   // break;
-                case "land_collision":
-                    return scriptEvents.land_collision;
-                   // break;
-                case "land_collision_end":
-                    return scriptEvents.land_collision_end;
-                   // break;
-                case "land_collision_start":
-                    return scriptEvents.land_collision_start;
-                   // break;
-                //case "link_message":
-                    //return scriptEvents.link_message;
-                  //  break;
-                case "listen":
-                    return scriptEvents.listen;
-                  //  break;
-                case "money":
-                    return scriptEvents.money;
-                   // break;
-                case "moving_end":
-                    return scriptEvents.moving_end;
-                   // break;
-                case "moving_start":
-                    return scriptEvents.moving_start;
-                   // break;
-                case "not_at_rot_target":
-                    return scriptEvents.not_at_rot_target;
-                   // break;
-                case "not_at_target":
-                    return scriptEvents.not_at_target;
-                  //  break;
-               // case "no_sensor":
-                    //return (long)scriptEvents.no_sensor;
-                    //break;
-                //case "on_rez":
-                    //return (long)scriptEvents.on_rez;
-                  //  break;
-                case "remote_data":
-                    return scriptEvents.remote_data;
-                   // break;
-                case "run_time_permissions":
-                    return scriptEvents.run_time_permissions;
-                   // break;
-                //case "sensor":
-                    //return (long)scriptEvents.sensor;
-                  //  break;
-                case "state_entry":
-                    return scriptEvents.state_entry;
-                   // break;
-                case "state_exit":
-                    return scriptEvents.state_exit;
-                   // break;
-                case "timer":
-                    return scriptEvents.timer;
-                   // break;
-                case "touch":
-                    return scriptEvents.touch;
-                   // break;
-                case "touch_end":
-                    return scriptEvents.touch_end;
-                   // break;
-                case "touch_start":
-                    return scriptEvents.touch_start;
-                   // break;
-                case "object_rez":
-                    return scriptEvents.object_rez;
-                default:
-                    return 0;
-                    //break;
-            }
-            //return 0;
-        }
+        
     }
 
-    [Flags]
-    public enum scriptEvents : int
-    {
-        None = 0,
-        attach = 1,
-        collision = 15,
-        collision_end = 32,
-        collision_start = 64,
-        control = 128,
-        dataserver = 256,
-        email = 512,
-        http_response = 1024,
-        land_collision = 2048,
-        land_collision_end = 4096,
-        land_collision_start = 8192,
-        at_target = 16384,
-        listen = 32768,
-        money = 65536,
-        moving_end = 131072,
-        moving_start = 262144,
-        not_at_rot_target = 524288,
-        not_at_target = 1048576,
-        remote_data = 8388608,
-        run_time_permissions = 268435456,
-        state_entry = 1073741824,
-        state_exit = 2,
-        timer = 4,
-        touch = 8,
-        touch_end = 536870912,
-        touch_start = 2097152,
-        object_rez = 4194304
-    }
-}
\ No newline at end of file
+    
+}