From f93fdfa5af0a8dd39f471a97a55209451f7ca7a0 Mon Sep 17 00:00:00 2001
From: Adam Frisby
Date: Wed, 6 Feb 2008 15:30:02 +0000
Subject: * Marking off which uses of catch (Exception e) is legit and which
 are not, for later removal/replacement.

---
 OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/Compiler.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'OpenSim/Region/ScriptEngine/DotNetEngine/Compiler')

diff --git a/OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/Compiler.cs b/OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/Compiler.cs
index 2f985e4..10959c5 100644
--- a/OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/Compiler.cs
+++ b/OpenSim/Region/ScriptEngine/DotNetEngine/Compiler/LSL/Compiler.cs
@@ -318,7 +318,7 @@ namespace OpenSim.Region.ScriptEngine.DotNetEngine.Compiler.LSL
             {
                 File.Delete(OutFile);
             }
-            catch (Exception e)
+            catch (Exception e) // NOTLEGIT - Should be just catching FileIOException
             {
                 //m_scriptEngine.Log.Error("[" + m_scriptEngine.ScriptEngineName + "]: Unable to delete old existring script-file before writing new. Compile aborted: " + e.ToString());
                 throw new Exception("Unable to delete old existring script-file before writing new. Compile aborted: " + e.ToString());
@@ -335,7 +335,7 @@ namespace OpenSim.Region.ScriptEngine.DotNetEngine.Compiler.LSL
                         Path.Combine("ScriptEngines", srcFileName),
                         Script);
                 }
-                catch (Exception ex)
+                catch (Exception ex) // NOTLEGIT - Should be just catching FileIOException
                 {
                     m_scriptEngine.Log.Error("[" + m_scriptEngine.ScriptEngineName + "]: Exception while trying to write script source to file \"" + srcFileName + "\": " + ex.ToString());
                 }
-- 
cgit v1.1