Changeset 387 for trunk/org.modelica.mdt.omc/src/org

Timestamp:

03/01/06 09:36:39 (19 years ago)

Author:

boris

Message:

• added a check in the error message parsing code makeing it more robust (in the face of omc wierdness)

File:

• trunk/org.modelica.mdt.omc/src/org/modelica/mdt/omc/internal/OMCParser.java (modified) (2 diffs)

trunk/org.modelica.mdt.omc/src/org/modelica/mdt/omc/internal/OMCParser.java

@@ -113 +113 @@
             
             /*
+             * we expect 5 (on unix) or 6 (on windows) ':' separated parts in a 
+             * proper error location string 
+             */ 
+            if (errorLocationParts.length < 5 || 
+                errorLocationParts.length > 6 )
+            {
+                throw new UnexpectedReplyException("Weird error message from "+
+                        "the compiler: [" + errorLine + "]");               
+            }
+            /*
              * This is the ugliest hack in a long while. Aaaah, nice.
              * 
@@ -119 +129 @@
              *   a separator in error messages, where the line and column
              *   numbers are found varies. If on Windows, the info we want
-             *   starts at array index 2, and on more normal systems, it starts
+             *   starts at array index 2, and on Unix, it starts
              *   at array index 1. Simply check if the element at index 1 is
              *   a digit or something else. Use this to set where in the array