Changed some ifs.

Author: KN4CK3R

AddressParser/Interpreter.cs

@@ -14,43 +14,56 @@ public IntPtr Execute(Operation operation, RemoteProcess process)
 			Contract.Requires(operation != null);
 			Contract.Requires(process != null);
 
-			if (operation is OffsetOperation)
+			var offsetOperation = operation as OffsetOperation;
+			if (offsetOperation != null)
 			{
-				return ((OffsetOperation)operation).Value;
+				return offsetOperation.Value;
 			}
-			else if (operation is ModuleOffsetOperation)
+
+			var moduleOffsetOperation = operation as ModuleOffsetOperation;
+			if (moduleOffsetOperation != null)
 			{
-				var module = process.GetModuleByName(((ModuleOffsetOperation)operation).Name);
+				var module = process.GetModuleByName(moduleOffsetOperation.Name);
 				if (module != null)
 				{
 					return module.Start;
 				}
 
 				return IntPtr.Zero;
 			}
-			else if (operation is AdditionOperation)
+
+			var additionOperation = operation as AdditionOperation;
+			if (additionOperation != null)
 			{
-				var addition = (AdditionOperation)operation;
+				var addition = additionOperation;
 				return Execute(addition.Argument1, process).Add(Execute(addition.Argument2, process));
 			}
-			else if (operation is SubtractionOperation)
+
+			var subtractionOperation = operation as SubtractionOperation;
+			if (subtractionOperation != null)
 			{
-				var addition = (SubtractionOperation)operation;
+				var addition = subtractionOperation;
 				return Execute(addition.Argument1, process).Sub(Execute(addition.Argument2, process));
 			}
-			else if (operation is MultiplicationOperation)
+
+			var multiplicationOperation = operation as MultiplicationOperation;
+			if (multiplicationOperation != null)
 			{
-				var multiplication = (MultiplicationOperation)operation;
+				var multiplication = multiplicationOperation;
 				return Execute(multiplication.Argument1, process).Mul(Execute(multiplication.Argument2, process));
 			}
-			else if (operation is DivisionOperation)
+
+			var divisionOperation = operation as DivisionOperation;
+			if (divisionOperation != null)
 			{
-				var division = (DivisionOperation)operation;
+				var division = divisionOperation;
 				return Execute(division.Dividend, process).Div(Execute(division.Divisor, process));
 			}
-			else if (operation is ReadPointerOperation)
+
+			var pointerOperation = operation as ReadPointerOperation;
+			if (pointerOperation != null)
 			{
-				return process.ReadRemoteObject<IntPtr>(Execute(((ReadPointerOperation)operation).Argument, process));
+				return process.ReadRemoteObject<IntPtr>(Execute(pointerOperation.Argument, process));
 			}
 
 			throw new ArgumentException($"Unsupported operation '{operation.GetType().FullName}'.");

DataExchange/ReClassFile.cs

@@ -106,7 +106,7 @@ private string TransformAddressString(string address)
 					part = part.Substring(1);
 				}
 
-				if (part.Contains(".exe") || part.Contains(".dll"))
+				if (isModule)
 				{
 					part = $"<{part}>";
 				}

UI/SettingsTextBox.cs

@@ -29,9 +29,10 @@ private void ReadSetting()
 			if (property != null && source != null)
 			{
 				var value = property.GetValue(source);
-				if (value is string)
+				var s = value as string;
+				if (s != null)
 				{
-					Text = (string)value;
+					Text = s;
 				}
 			}
 		}

Util/Rtf/RtfBuilder.cs

@@ -209,7 +209,7 @@ protected void AppendInternal(string value)
 				using (new RtfFormatWrapper(this))
 				{
 					value = EscapeString(value);
-					if (value.IndexOf(Environment.NewLine) >= 0)
+					if (value.IndexOf(Environment.NewLine, StringComparison.Ordinal) >= 0)
 					{
 						var lines = value.Split(new[] { Environment.NewLine }, StringSplitOptions.None);