Updates after resharper inspection.

Author: Oren (electricessence)

ActionRunner.cs

@@ -25,47 +25,27 @@ public static ActionRunner Create<T>(Func<T> action, TaskScheduler scheduler = n
 		}
 
 		Action _action;
+		// ReSharper disable once NotAccessedField.Global
 		protected TaskScheduler _scheduler;
 
 		protected int _count;
-		public int Count
-		{
-			get { return _count; }
-		}
+		public int Count => _count;
 
 		public DateTime LastStart
 		{
 			get;
 			protected set;
 		}
 
-		public bool HasBeenRun
-		{
-			get
-			{
-				return LastStart < DateTime.Now;
-			}
-		}
+		public bool HasBeenRun => LastStart < DateTime.Now;
 
 		public DateTime LastComplete
 		{
 			get;
 			protected set;
 		}
 
-		public bool HasCompleted
-		{
-			get
-			{
-				return LastComplete < DateTime.Now;
-			}
-		}
-
-		public Exception LastFault
-		{
-			get;
-			protected set;
-		}
+		public bool HasCompleted => LastComplete < DateTime.Now;
 
 		public bool Cancel(bool onlyIfNotRunning)
 		{
@@ -84,6 +64,7 @@ public void Dispose()
 		{
 			Cancel();
 			_action = null;
+			_scheduler = null;
 		}
 
 		Action GetAction()
@@ -94,13 +75,7 @@ Action GetAction()
 			return a;
 		}
 
-		public bool IsScheduled
-		{
-			get
-			{
-				return _task?.IsActive() ?? false;
-			}
-		}
+		public bool IsScheduled => _task?.IsActive() ?? false;
 
 		/// <summary>
 		/// Indiscriminately invokes the action.
@@ -110,6 +85,7 @@ public void RunSynchronously()
 			GetAction().Invoke();
 		}
 
+		// ReSharper disable once UnusedMember.Local
 		readonly object _taskLock = new object();
 		CancellableTask _task;
 		CancellableTask Prepare()
@@ -119,7 +95,6 @@ CancellableTask Prepare()
 			task
 				.OnFaulted(ex =>
 				{
-					LastFault = ex;
 				})
 				.OnFullfilled(() =>
 				{
@@ -145,14 +120,12 @@ public CancellableTask Defer(TimeSpan delay, bool clearSchedule = true)
 				Cancel(true); // Don't cancel defered if already running.
 			}
 
-			CancellableTask task = null;
-			if ((task = _task) == null)
+			CancellableTask task;
+			if ((task = _task) != null) return task;
+			task = Prepare();
+			if (null == Interlocked.CompareExchange(ref _task, task, null))
 			{
-				task = Prepare();
-				if (null == Interlocked.CompareExchange(ref _task, task, null))
-				{
-					task.Start(delay);
-				}
+				task.Start(delay);
 			}
 			return task;
 		}

CancellableTask.cs

@@ -84,7 +84,7 @@ public void Start(TimeSpan delay, TaskScheduler scheduler = null)
 			}
 			else
 			{
-				int runState = 0;
+				var runState = 0;
 
 				ContinueWith(t =>
 				{

ICancellable.cs

@@ -1,7 +1,9 @@
 using System;
+using System.Diagnostics.CodeAnalysis;
 
 namespace Open.Threading.Tasks
 {
+	[SuppressMessage("ReSharper", "UnusedMemberInSuper.Global")]
 	public interface ICancellable : IDisposable
 	{
 		/// <summary>

Open.Threading.Tasks.csproj

@@ -14,14 +14,19 @@ Part of the "Open" set of libraries.</Description>
     <PackageLicenseUrl>https://github.com/electricessence/Open.Threading.Tasks/blob/master/LISCENSE.md</PackageLicenseUrl>
     <RepositoryType>git</RepositoryType>
     <PackageTags>dotnet, dotnet-core, dotnetcore, cs, extensions, actionrunner, cancellable, cancellabletask, progress, task-extensions</PackageTags>
-    <Version>1.1.2</Version>
+    <Version>1.1.3</Version>
     <AssemblyVersion>1.1.2.0</AssemblyVersion>
     <FileVersion>1.1.2.0</FileVersion>
     <PackageReleaseNotes></PackageReleaseNotes>
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>True</Optimize>
+    <LangVersion>latest</LangVersion>
+  </PropertyGroup>
+
+  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|AnyCPU'">
+    <LangVersion>latest</LangVersion>
   </PropertyGroup>
 
   <ItemGroup>

TaskExtensions.cs

@@ -33,29 +33,28 @@ public static bool IsActive(this Task target)
 		/// <summary>
 		/// Checks the status of the task and attempts to start it if waiting to start (TaskStatus.Created).
 		/// </summary>
+		/// <param name="target">The task to ensure start.</param>
 		/// <param name="scheduler">Optional scheduler to use.</param>
 		/// <returns>True if start attempt was successful.</returns>
 		public static bool EnsureStarted(this Task target, TaskScheduler scheduler = null)
 		{
 			if (target == null) throw new NullReferenceException();
 
-			if (target.Status == TaskStatus.Created)
+			if (target.Status != TaskStatus.Created) return false;
+			try
 			{
-				try
-				{
-					if (scheduler == null)
-						target.Start();
-					else
-						target.Start(scheduler);
+				if (scheduler == null)
+					target.Start();
+				else
+					target.Start(scheduler);
 
-					return true;
-				}
-				catch (InvalidOperationException)
-				{
-					// Even though we've checked the status, it's possible it could have been started.  We can't guarantee proper handling without a trap here.
-					if (target.Status == TaskStatus.Created)
-						throw; // Something wierd must have happened if we arrived here.
-				}
+				return true;
+			}
+			catch (InvalidOperationException)
+			{
+				// Even though we've checked the status, it's possible it could have been started.  We can't guarantee proper handling without a trap here.
+				if (target.Status == TaskStatus.Created)
+					throw; // Something wierd must have happened if we arrived here.
 			}
 
 			return false;
@@ -65,6 +64,7 @@ public static bool EnsureStarted(this Task target, TaskScheduler scheduler = nul
 		/// Utility method that can be chained with other methods for reacting to Task results.  Only invokes the action if completed and not cancelled.
 		/// </summary>
 		/// <typeparam name="TTask">The return type is the same as the target.</typeparam>
+		/// <param name="target">The task.</param>
 		/// <param name="action">The action to perform if fullfulled.</param>
 		/// <returns>The target object.  Allows for method chaining.</returns>
 		public static TTask OnFullfilled<TTask>(this TTask target, Action action)
@@ -81,7 +81,8 @@ public static TTask OnFullfilled<TTask>(this TTask target, Action action)
 		/// <summary>
 		/// Utility method that can be chained with other methods for reacting to Task results.  Only invokes the action if completed and not cancelled.
 		/// </summary>
-		/// <typeparam name="TTask">The return type is the same as the target.</typeparam>
+		/// <typeparam name="T">The return type is the same as the target.</typeparam>
+		/// <param name="target">The task.</param>
 		/// <param name="action">The action to perform if fullfulled.</param>
 		/// <returns>The target object.  Allows for method chaining.</returns>
 		public static Task<T> OnFullfilled<T>(this Task<T> target, Action<T> action)
@@ -97,7 +98,9 @@ public static Task<T> OnFullfilled<T>(this Task<T> target, Action<T> action)
 		/// <summary>
 		/// Utility method that can be chained with other methods for reacting to Task results.  Only invokes the action if completed and not cancelled.
 		/// </summary>
-		/// <typeparam name="TTask">The return type is the same as the target.</typeparam>
+		/// <typeparam name="TTask">The task type.</typeparam>
+		/// <typeparam name="T">The return type of the task.</typeparam>
+		/// <param name="target">The task.</param>
 		/// <param name="action">The action to perform if fullfulled.</param>
 		/// <returns>The target object.  Allows for method chaining.</returns>
 		public static TTask OnFullfilled<TTask, T>(this TTask target, Func<T> action)
@@ -115,6 +118,7 @@ public static TTask OnFullfilled<TTask, T>(this TTask target, Func<T> action)
 		/// Utility method that can be chained with other methods for reacting to Task results. Only invokes the action if faulted.
 		/// </summary>
 		/// <typeparam name="TTask">The return type is the same as the target.</typeparam>
+		/// <param name="target">The task.</param>
 		/// <param name="action">The action to perform if faulted.</param>
 		/// <returns>The target object.  Allows for method chaining.</returns>
 		public static TTask OnFaulted<TTask>(this TTask target, Action<Exception> action)
@@ -133,6 +137,7 @@ public static TTask OnFaulted<TTask>(this TTask target, Action<Exception> action
 		/// Utility method that can be chained with other methods for reacting to Task results.  Only invokes the action if cancelled.
 		/// </summary>
 		/// <typeparam name="TTask">The return type is the same as the target.</typeparam>
+		/// <param name="target">The task.</param>
 		/// <param name="action">The action to perform if cancelled.</param>
 		/// <returns>The target object.  Allows for method chaining.</returns>
 		public static TTask OnCancelled<TTask>(this TTask target, Action action)
@@ -149,7 +154,9 @@ public static TTask OnCancelled<TTask>(this TTask target, Action action)
 		/// <summary>
 		/// Utility method that can be chained with other methods for reacting to Task results.  Only invokes the action if cancelled.
 		/// </summary>
-		/// <typeparam name="TTask">The return type is the same as the target.</typeparam>
+		/// <typeparam name="TTask">The task type.</typeparam>
+		/// <typeparam name="T">The return type of the task.</typeparam>
+		/// <param name="target">The task.</param>
 		/// <param name="action">The action to perform if cancelled.</param>
 		/// <returns>The target object.  Allows for method chaining.</returns>
 		public static TTask OnCancelled<TTask, T>(this TTask target, Func<T> action)

TimeoutHandler.cs

@@ -6,8 +6,7 @@ namespace Open.Threading.Tasks
 {
 	public class TimeoutHandler : IDisposable
 	{
-
-		CancellationTokenSource TokenSource;
+		readonly CancellationTokenSource TokenSource;
 		TimeoutHandler(int delay, Action<int> onComplete)
 		{
 			TokenSource = new CancellationTokenSource();