Was made refactoring.

Author: Taritsyn

src/MsieJavaScriptEngine/ActiveScript/ActiveScriptJsEngineBase.cs

@@ -83,7 +83,7 @@ internal abstract class ActiveScriptJsEngineBase : IInnerJsEngine, IActiveScript
 		/// <summary>
 		/// Flag that object is destroyed
 		/// </summary>
-		private bool _disposed;
+		private StatedFlag _disposedFlag = new StatedFlag();
 
 
 		/// <summary>
@@ -532,10 +532,8 @@ private void Dispose(bool disposing)
 		{
 			_dispatcher.Invoke(DispatcherPriority.Input, (Action)(() =>
 			{
-				if (!_disposed)
+				if (_disposedFlag.Set())
 				{
-					_disposed = true;
-
 					if (_dispatch != null)
 					{
 						ComHelpers.ReleaseComObject(ref _dispatch, !disposing);

src/MsieJavaScriptEngine/ActiveScript/ActiveScriptParseWrapper.cs

@@ -42,7 +42,7 @@ internal sealed class ActiveScriptParseWrapper : IActiveScriptParseWrapper
 		/// <summary>
 		/// Flag that object is destroyed
 		/// </summary>
-		private bool _disposed;
+		private StatedFlag _disposedFlag = new StatedFlag();
 
 		/// <summary>
 		/// Gets a last COM exception
@@ -81,7 +81,7 @@ public ActiveScriptParseWrapper(IntPtr pActiveScript, IActiveScript activeScript
 		}
 
 		/// <summary>
-		/// Destructs instance of <see cref="ActiveScriptSiteWrapper"/>
+		/// Destructs instance of <see cref="ActiveScriptParseWrapper"/>
 		/// </summary>
 		~ActiveScriptParseWrapper()
 		{
@@ -96,10 +96,8 @@ public ActiveScriptParseWrapper(IntPtr pActiveScript, IActiveScript activeScript
 		/// managed objects contained in fields of class</param>
 		private void Dispose(bool disposing)
 		{
-			if (!_disposed)
+			if (_disposedFlag.Set())
 			{
-				_disposed = true;
-
 				if (_is64Bit)
 				{
 					_activeScriptParse64 = null;

src/MsieJavaScriptEngine/InterlockedStatedFlag.cs

@@ -0,0 +1,20 @@
+namespace MsieJavaScriptEngine
+{
+	using System.Threading;
+
+	internal struct InterlockedStatedFlag
+	{
+		private int _counter;
+
+
+		public bool IsSet()
+		{
+			return _counter != 0;
+		}
+
+		public bool Set()
+		{
+			return Interlocked.Exchange(ref _counter, 1) == 0;
+		}
+	}
+}

src/MsieJavaScriptEngine/JsRt/ChakraJsRtJsEngineBase.cs

@@ -27,7 +27,7 @@ internal abstract class ChakraJsRtJsEngineBase : IInnerJsEngine
 		/// <summary>
 		/// Flag indicating whether debugging started
 		/// </summary>
-		private bool _debuggingStarted;
+		private StatedFlag _debuggingStartedFlag;
 
 
 		/// <summary>
@@ -48,10 +48,9 @@ protected ChakraJsRtJsEngineBase(JsEngineMode engineMode, bool enableDebugging)
 		/// </summary>
 		protected void StartDebugging()
 		{
-			if (!_debuggingStarted)
+			if (_debuggingStartedFlag.Set())
 			{
 				InnerStartDebugging();
-				_debuggingStarted = true;
 			}
 		}
 

src/MsieJavaScriptEngine/JsRt/Edge/ChakraEdgeJsRtJsEngine.cs

@@ -41,7 +41,7 @@ internal sealed class ChakraEdgeJsRtJsEngine : ChakraJsRtJsEngineBase
 		/// <summary>
 		/// Flag that object is destroyed
 		/// </summary>
-		private bool _disposed;
+		private StatedFlag _disposedFlag = new StatedFlag();
 
 
 		/// <summary>
@@ -379,10 +379,8 @@ private void Dispose(bool disposing)
 		{
 			lock (_runSynchronizer)
 			{
-				if (!_disposed)
+				if (_disposedFlag.Set())
 				{
-					_disposed = true;
-
 					_jsRuntime.Dispose();
 				}
 			}

src/MsieJavaScriptEngine/JsRt/Edge/EdgeJsScope.cs

@@ -16,7 +16,7 @@ internal struct EdgeJsScope : IDisposable
 		/// <summary>
 		/// Whether the structure has been disposed
 		/// </summary>
-		private bool _disposed;
+		private StatedFlag _disposedFlag;
 
 
 		/// <summary>
@@ -25,7 +25,7 @@ internal struct EdgeJsScope : IDisposable
 		/// <param name="context">The context to create the scope for</param>
 		public EdgeJsScope(EdgeJsContext context)
 		{
-			_disposed = false;
+			_disposedFlag = new StatedFlag();
 			_previousContext = EdgeJsContext.Current;
 			EdgeJsContext.Current = context;
 		}
@@ -37,13 +37,10 @@ public EdgeJsScope(EdgeJsContext context)
 		/// </summary>
 		public void Dispose()
 		{
-			if (_disposed)
+			if (_disposedFlag.Set())
 			{
-				return;
+				EdgeJsContext.Current = _previousContext;
 			}
-
-			EdgeJsContext.Current = _previousContext;
-			_disposed = true;
 		}
 
 		#endregion

src/MsieJavaScriptEngine/JsRt/Ie/ChakraIeJsRtJsEngine.cs

@@ -46,7 +46,7 @@ internal sealed class ChakraIeJsRtJsEngine : ChakraJsRtJsEngineBase
 		/// <summary>
 		/// Flag that object is destroyed
 		/// </summary>
-		private bool _disposed;
+		private StatedFlag _disposedFlag = new StatedFlag();
 
 
 		/// <summary>
@@ -415,10 +415,8 @@ private void Dispose(bool disposing)
 		{
 			lock (_runSynchronizer)
 			{
-				if (!_disposed)
+				if (_disposedFlag.Set())
 				{
-					_disposed = true;
-
 					_jsRuntime.Dispose();
 				}
 			}

src/MsieJavaScriptEngine/JsRt/Ie/IeJsScope.cs

@@ -16,7 +16,7 @@ internal struct IeJsScope : IDisposable
 		/// <summary>
 		/// Whether the structure has been disposed
 		/// </summary>
-		private bool _disposed;
+		private StatedFlag _disposedFlag;
 
 
 		/// <summary>
@@ -25,7 +25,7 @@ internal struct IeJsScope : IDisposable
 		/// <param name="context">The context to create the scope for</param>
 		public IeJsScope(IeJsContext context)
 		{
-			_disposed = false;
+			_disposedFlag = new StatedFlag();
 			_previousContext = IeJsContext.Current;
 			IeJsContext.Current = context;
 		}
@@ -37,13 +37,10 @@ public IeJsScope(IeJsContext context)
 		/// </summary>
 		public void Dispose()
 		{
-			if (_disposed)
+			if (_disposedFlag.Set())
 			{
-				return;
+				IeJsContext.Current = _previousContext;
 			}
-
-			IeJsContext.Current = _previousContext;
-			_disposed = true;
 		}
 
 		#endregion

src/MsieJavaScriptEngine/MsieJavaScriptEngine.csproj

@@ -76,6 +76,7 @@
     <Compile Include="HostObject.cs" />
     <Compile Include="HostType.cs" />
     <Compile Include="IInnerJsEngine.cs" />
+    <Compile Include="InterlockedStatedFlag.cs" />
     <Compile Include="JsEngineSettings.cs" />
     <Compile Include="JsRt\ChakraJsRtJsEngineBase.cs" />
     <Compile Include="JsRt\Edge\ChakraEdgeJsRtJsEngine.cs" />
@@ -142,6 +143,7 @@
       <DesignTime>True</DesignTime>
       <DependentUpon>Strings.ru-ru.resx</DependentUpon>
     </Compile>
+    <Compile Include="StatedFlag.cs" />
     <Compile Include="Utilities\TypeConverter.cs" />
     <Compile Include="Undefined.cs" />
     <Compile Include="Utilities\StringBuilderExtensions.cs" />

src/MsieJavaScriptEngine/MsieJsEngine.cs

@@ -34,7 +34,7 @@ public sealed class MsieJsEngine : IDisposable
 		/// <summary>
 		/// Flag that object is destroyed
 		/// </summary>
-		private bool _disposed;
+		private InterlockedStatedFlag _disposedFlag = new InterlockedStatedFlag();
 
 		/// <summary>
 		/// Gets a name of JavaScript engine mode
@@ -173,7 +173,7 @@ public MsieJsEngine(JsEngineSettings settings)
 
 		private void VerifyNotDisposed()
 		{
-			if (_disposed)
+			if (_disposedFlag.IsSet())
 			{
 				throw new ObjectDisposedException(ToString());
 			}
@@ -718,10 +718,8 @@ public void EmbedHostType(string itemName, Type type)
 		/// </summary>
 		public void Dispose()
 		{
-			if (!_disposed)
+			if (_disposedFlag.Set())
 			{
-				_disposed = true;
-
 				if (_jsEngine != null)
 				{
 					_jsEngine.Dispose();