Small changes.

Author: KN4CK3R

AddressParser/AstBuilder.cs

@@ -89,12 +89,11 @@ public Operation Build(IEnumerable<Token> tokens)
 
 		private void PopOperations(bool untillLeftBracket)
 		{
-			Operation lastOperation = null;
 			while (operatorStack.Count > 0 && operatorStack.Peek().TokenType != TokenType.LeftBracket)
 			{
 				var token = operatorStack.Pop();
 
-				lastOperation = ConvertOperation(token);
+				var lastOperation = ConvertOperation(token);
 
 				resultStack.Push(lastOperation);
 			}

CodeGenerator/CppCodeGenerator.cs

@@ -76,7 +76,7 @@ public string GenerateCode(IEnumerable<ClassNode> classes, ILogger logger)
 							string.Join(
 								"\n",
 								YieldMemberDefinitions(c.Nodes.Skip(skipFirstMember ? 1 : 0).WhereNot(n => n is FunctionNode), logger)
-									.Select(m => MemberDefinitionToString(m))
+									.Select(MemberDefinitionToString)
 									.Select(s => "\t" + s)
 							)
 						);

DataExchange/ReClassFile.cs

@@ -169,7 +169,7 @@ private IEnumerable<BaseNode> ReadNodeElements(IEnumerable<XElement> elements, C
 
 					while (size != 0)
 					{
-						BaseNode paddingNode = null;
+						BaseNode paddingNode;
 #if WIN64
 						if (size >= 8)
 						{
@@ -203,7 +203,7 @@ private IEnumerable<BaseNode> ReadNodeElements(IEnumerable<XElement> elements, C
 				var referenceNode = node as BaseReferenceNode;
 				if (referenceNode != null)
 				{
-					string reference = null;
+					string reference;
 					if (referenceNode is ClassInstanceArrayNode)
 					{
 						reference = element.Element("Array")?.Attribute("Name")?.Value;

Forms/LogForm.cs

@@ -73,10 +73,7 @@ private void showDetailsToolStripMenuItem_Click(object sender, EventArgs e)
 		private void RefreshDataBinding()
 		{
 			var cm = entriesDataGridView.BindingContext[items] as CurrencyManager;
-			if (cm != null)
-			{
-				cm.Refresh();
-			}
+			cm?.Refresh();
 		}
 
 		public void Clear()
@@ -115,12 +112,9 @@ public void Add(LogLevel level, string message, Exception ex)
 		private void ShowDetailsForm()
 		{
 			var item = entriesDataGridView.SelectedRows.Cast<DataGridViewRow>().FirstOrDefault()?.DataBoundItem as LogItem;
-			if (item != null)
+			if (item?.Exception != null)
 			{
-				if (item.Exception != null)
-				{
-					Program.ShowException(item.Exception);
-				}
+				Program.ShowException(item.Exception);
 			}
 		}
 	}

Forms/ProcessBrowserForm.cs

@@ -25,15 +25,9 @@ public partial class ProcessBrowserForm : IconForm
 		private readonly CoreFunctionsManager coreFunctions;
 
 		/// <summary>Gets the selected process.</summary>
-		public ProcessInfo SelectedProcess
-		{
-			get
-			{
-				return (processDataGridView.SelectedRows.Cast<DataGridViewRow>().FirstOrDefault()?.DataBoundItem as DataRowView)
-					?.Row
-					?.Field<ProcessInfo>("info");
-			}
-		}
+		public ProcessInfo SelectedProcess => (processDataGridView.SelectedRows.Cast<DataGridViewRow>().FirstOrDefault()?.DataBoundItem as DataRowView)
+			?.Row
+			?.Field<ProcessInfo>("info");
 
 		/// <summary>Gets if symbols should get loaded.</summary>
 		public bool LoadSymbols => loadSymbolsCheckBox.Checked;

Forms/ProcessInfoForm.cs

@@ -170,8 +170,8 @@ private void createClassAtAddressToolStripMenuItem_Click(object sender, EventArg
 		private void dumpToolStripMenuItem_Click(object sender, EventArgs e)
 		{
 			bool isModule;
-			string fileName = string.Empty;
-			string initialDirectory = string.Empty;
+			string fileName;
+			var initialDirectory = string.Empty;
 			IntPtr address;
 			int size;
 

Memory/Disassembler.cs

@@ -57,7 +57,7 @@ public IEnumerable<DisassembledInstruction> DisassembleCode(IntPtr address, int
 			var eip = address;
 			var end = address + length;
 
-			var instruction = new InstructionData();
+			InstructionData instruction;
 			while (eip.CompareTo(end) == -1)
 			{
 				var res = coreFunctions.DisassembleCode(eip, end.Sub(eip).ToInt32() + 1, virtualAddress, out instruction);

Memory/RemoteProcess.cs

@@ -118,7 +118,7 @@ public void Close()
 
 		/// <summary>Reads remote memory from the address into the buffer.</summary>
 		/// <param name="address">The address to read from.</param>
-		/// <param name="data">[out] The data buffer to fill. If the remote process is not valid, the buffer will get filled with zeros.</param>
+		/// <param name="buffer">[out] The data buffer to fill. If the remote process is not valid, the buffer will get filled with zeros.</param>
 		public bool ReadRemoteMemoryIntoBuffer(IntPtr address, ref byte[] buffer)
 		{
 			Contract.Requires(buffer != null);
@@ -129,7 +129,7 @@ public bool ReadRemoteMemoryIntoBuffer(IntPtr address, ref byte[] buffer)
 
 		/// <summary>Reads remote memory from the address into the buffer.</summary>
 		/// <param name="address">The address to read from.</param>
-		/// <param name="data">[out] The data buffer to fill. If the remote process is not valid, the buffer will get filled with zeros.</param>
+		/// <param name="buffer">[out] The data buffer to fill. If the remote process is not valid, the buffer will get filled with zeros.</param>
 		/// <param name="offset">The offset in the data.</param>
 		/// <param name="length">The number of bytes to read.</param>
 		public bool ReadRemoteMemoryIntoBuffer(IntPtr address, ref byte[] buffer, int offset, int length)
@@ -255,7 +255,7 @@ public string ReadRemoteRuntimeTypeInformation(IntPtr address)
 		{
 			if (address.MayBeValid())
 			{
-				string rtti = null;
+				string rtti;
 				if (!rttiCache.TryGetValue(address, out rtti))
 				{
 					var objectLocatorPtr = ReadRemoteObject<IntPtr>(address - IntPtr.Size);

Plugins/PluginManager.cs

@@ -69,7 +69,7 @@ private void LoadPlugins(IEnumerable<FileInfo> files, ILogger logger)
 
 			foreach (var fi in files)
 			{
-				FileVersionInfo fvi = null;
+				FileVersionInfo fvi;
 				try
 				{
 					fvi = FileVersionInfo.GetVersionInfo(fi.FullName);

Program.cs

@@ -12,26 +12,20 @@ namespace ReClassNET
 {
 	static class Program
 	{
-		private static Settings settings;
-		private static ILogger logger;
-		private static Random random = new Random();
-		private static MainForm mainForm;
-		private static bool designMode = true;
+		public static Settings Settings { get; private set; }
 
-		public static Settings Settings => settings;
+		public static ILogger Logger { get; private set; }
 
-		public static ILogger Logger => logger;
+		public static Random GlobalRandom { get; } = new Random();
 
-		public static Random GlobalRandom => random;
+		public static MainForm MainForm { get; private set; }
 
-		public static MainForm MainForm => mainForm;
-
-		public static bool DesignMode => designMode;
+		public static bool DesignMode { get; private set; } = true;
 
 		[STAThread]
 		static void Main()
 		{
-			designMode = false; // The designer doesn't call Main()
+			DesignMode = false; // The designer doesn't call Main()
 
 			DpiUtil.ConfigureProcess();
 
@@ -42,14 +36,14 @@ static void Main()
 
 			CultureInfo.DefaultThreadCurrentCulture = CultureInfo.InvariantCulture;
 
-			settings = Settings.Load(Constants.SettingsFile);
-			logger = new GuiLogger();
+			Settings = Settings.Load(Constants.SettingsFile);
+			Logger = new GuiLogger();
 #if DEBUG
 			using (var coreFunctions = new CoreFunctionsManager())
 			{
-				mainForm = new MainForm(coreFunctions);
+				MainForm = new MainForm(coreFunctions);
 
-				Application.Run(mainForm);
+				Application.Run(MainForm);
 			}
 #else
 			try
@@ -67,7 +61,7 @@ static void Main()
 			}
 #endif
 
-			Settings.Save(settings, Constants.SettingsFile);
+			Settings.Save(Settings, Constants.SettingsFile);
 		}
 
 		/// <summary>Shows the exception in a special form.</summary>
@@ -76,9 +70,11 @@ public static void ShowException(Exception ex)
 		{
 			ex.HelpLink = Constants.HelpUrl;
 
-			var msg = new ExceptionMessageBox(ex);
-			msg.ShowToolBar = true;
-			msg.Symbol = ExceptionMessageBoxSymbol.Error;
+			var msg = new ExceptionMessageBox(ex)
+			{
+				ShowToolBar = true,
+				Symbol = ExceptionMessageBoxSymbol.Error
+			};
 			msg.Show(null);
 		}
 	}