In JsRT modes the compilation error messages now contains a information about the error location

Author: Taritsyn

NuGet/MsieJavaScriptEngine.nuspec

@@ -13,7 +13,8 @@
 		<description>This library is a .NET wrapper for working with the JavaScript engines of Internet Explorer and Edge (JsRT versions of Chakra, ActiveScript version of Chakra and Classic JavaScript Engine). Project was based on the code of SassAndCoffee.JavaScript (http://github.com/paulcbetts/SassAndCoffee), Chakra Sample Hosts (http://github.com/panopticoncentral/chakra-host) and jsrt-dotnet (http://github.com/robpaveza/jsrt-dotnet).</description>
 		<summary>This library is a .NET wrapper for working with the JavaScript engines of Internet Explorer and Edge (JsRT versions of Chakra, ActiveScript version of Chakra and Classic JavaScript Engine).</summary>
 		<releaseNotes>1. In ActiveScript modes now are uses the short names of error categories;
-2. In `Classic` mode during debugging now script error contains a full stack trace.</releaseNotes>
+2. In `Classic` mode during debugging now script error contains a full stack trace;
+3. In JsRT modes the compilation error messages now contains a information about the error location.</releaseNotes>
 		<copyright>Copyright (c) 2012-2017 Andrey Taritsyn - http://www.taritsyn.ru</copyright>
 		<language>en-US</language>
 		<tags>JavaScript ECMAScript MSIE IE Edge Chakra</tags>

NuGet/readme.txt

@@ -23,7 +23,9 @@
    =============
    1. In ActiveScript modes now are uses the short names of error categories;
    2. In `Classic` mode during debugging now script error contains a full stack
-      trace.
+      trace;
+   3. In JsRT modes the compilation error messages now contains a information about
+      the error location.
 
    ============
    PROJECT SITE

src/MsieJavaScriptEngine/JsRt/ChakraJsRtJsEngineBase.cs

@@ -3,6 +3,7 @@
 using System.Collections.Generic;
 using System.Runtime.InteropServices;
 #endif
+using System.Text;
 using System.Text.RegularExpressions;
 
 namespace MsieJavaScriptEngine.JsRt
@@ -67,6 +68,39 @@ protected ChakraJsRtJsEngineBase(JsEngineMode engineMode, bool enableDebugging)
 		}
 
 
+		/// <summary>
+		/// Generates a error message with location
+		/// </summary>
+		/// <param name="category">Error category</param>
+		/// <param name="message">Error message</param>
+		/// <param name="lineNumber">Line number</param>
+		/// <param name="columnNumber">Column number</param>
+		/// <returns>Error message with location</returns>
+		protected static string GenerateErrorMessageWithLocation(string category, string message,
+			int lineNumber, int columnNumber)
+		{
+			var messageBuilder = new StringBuilder();
+			if (!string.IsNullOrWhiteSpace(category))
+			{
+				messageBuilder.AppendFormat("{0}: ", category);
+			}
+			messageBuilder.Append(message);
+			if (lineNumber > 0)
+			{
+				messageBuilder.AppendLine();
+				messageBuilder.AppendFormat("   at {0}", lineNumber);
+				if (columnNumber > 0)
+				{
+					messageBuilder.AppendFormat(":{0}", columnNumber);
+				}
+			}
+
+			string errorMessage = messageBuilder.ToString();
+			messageBuilder.Clear();
+
+			return errorMessage;
+		}
+
 		/// <summary>
 		/// Gets a error coordinates from message
 		/// </summary>

src/MsieJavaScriptEngine/JsRt/Edge/ChakraEdgeJsRtJsEngine.cs

@@ -827,22 +827,6 @@ private JsRuntimeException ConvertJsExceptionToJsRuntimeException(
 
 				if (errorValue.IsValid)
 				{
-					EdgeJsPropertyId stackPropertyId = EdgeJsPropertyId.FromString("stack");
-					if (errorValue.HasProperty(stackPropertyId))
-					{
-						EdgeJsValue stackPropertyValue = errorValue.GetProperty(stackPropertyId);
-						message = stackPropertyValue.ConvertToString().ToString();
-					}
-					else
-					{
-						EdgeJsValue messagePropertyValue = errorValue.GetProperty("message");
-						string scriptMessage = messagePropertyValue.ConvertToString().ToString();
-						if (!string.IsNullOrWhiteSpace(scriptMessage))
-						{
-							message = string.Format("{0}: {1}", message.TrimEnd('.'), scriptMessage);
-						}
-					}
-
 					EdgeJsPropertyId linePropertyId = EdgeJsPropertyId.FromString("line");
 					if (errorValue.HasProperty(linePropertyId))
 					{
@@ -857,6 +841,20 @@ private JsRuntimeException ConvertJsExceptionToJsRuntimeException(
 						columnNumber = columnPropertyValue.ConvertToNumber().ToInt32() + 1;
 					}
 
+					EdgeJsPropertyId stackPropertyId = EdgeJsPropertyId.FromString("stack");
+					if (errorValue.HasProperty(stackPropertyId))
+					{
+						EdgeJsValue stackPropertyValue = errorValue.GetProperty(stackPropertyId);
+						message = stackPropertyValue.ConvertToString().ToString();
+					}
+					else
+					{
+						EdgeJsValue messagePropertyValue = errorValue.GetProperty("message");
+						string scriptMessage = messagePropertyValue.ConvertToString().ToString();
+						message = GenerateErrorMessageWithLocation(message.TrimEnd('.'), scriptMessage,
+							lineNumber, columnNumber);
+					}
+
 					if (lineNumber <= 0 && columnNumber <= 0)
 					{
 						GetErrorCoordinatesFromMessage(message, out lineNumber, out columnNumber);

src/MsieJavaScriptEngine/JsRt/Ie/ChakraIeJsRtJsEngine.cs

@@ -850,34 +850,32 @@ private JsRuntimeException ConvertJsExceptionToJsRuntimeException(JsException js
 
 				if (errorValue.IsValid)
 				{
-					IeJsPropertyId stackPropertyId = IeJsPropertyId.FromString("stack");
-					if (errorValue.HasProperty(stackPropertyId))
-					{
-						IeJsValue stackPropertyValue = errorValue.GetProperty(stackPropertyId);
-						message = stackPropertyValue.ConvertToString().ToString();
-					}
-					else
-					{
-						IeJsValue messagePropertyValue = errorValue.GetProperty("message");
-						string scriptMessage = messagePropertyValue.ConvertToString().ToString();
-						if (!string.IsNullOrWhiteSpace(scriptMessage))
-						{
-							message = string.Format("{0}: {1}", message.TrimEnd('.'), scriptMessage);
-						}
-					}
-
 					IeJsPropertyId linePropertyId = IeJsPropertyId.FromString("line");
 					if (errorValue.HasProperty(linePropertyId))
 					{
 						IeJsValue linePropertyValue = errorValue.GetProperty(linePropertyId);
-						lineNumber = (int) linePropertyValue.ConvertToNumber().ToDouble() + 1;
+						lineNumber = (int)linePropertyValue.ConvertToNumber().ToDouble() + 1;
 					}
 
 					IeJsPropertyId columnPropertyId = IeJsPropertyId.FromString("column");
 					if (errorValue.HasProperty(columnPropertyId))
 					{
 						IeJsValue columnPropertyValue = errorValue.GetProperty(columnPropertyId);
-						columnNumber = (int) columnPropertyValue.ConvertToNumber().ToDouble() + 1;
+						columnNumber = (int)columnPropertyValue.ConvertToNumber().ToDouble() + 1;
+					}
+
+					IeJsPropertyId stackPropertyId = IeJsPropertyId.FromString("stack");
+					if (errorValue.HasProperty(stackPropertyId))
+					{
+						IeJsValue stackPropertyValue = errorValue.GetProperty(stackPropertyId);
+						message = stackPropertyValue.ConvertToString().ToString();
+					}
+					else
+					{
+						IeJsValue messagePropertyValue = errorValue.GetProperty("message");
+						string scriptMessage = messagePropertyValue.ConvertToString().ToString();
+						message = GenerateErrorMessageWithLocation(message.TrimEnd('.'), scriptMessage,
+							lineNumber, columnNumber);
 					}
 
 					if (lineNumber <= 0 && columnNumber <= 0)