* Clean all processes when using drag and drop

* Minor settings logic fix

Author: CodeDead

MemPlus/Business/LOG/LogController.cs

@@ -186,17 +186,20 @@ internal void SetSaveToFile(bool saveToFile)
                 DisposeFileResources();
             }
 
-            if (saveToFile)
+            if (_logPath != null && saveToFile && _logPath.Length > 0)
             {
                 // Generate a new FileStream that allows other handles to access the file
                 _fileStream = new FileStream(_logPath,
                     FileMode.Append,
                     FileAccess.Write,
                     FileShare.ReadWrite);
                 _streamWriter = new StreamWriter(_fileStream) {AutoFlush = true};
+                _saveToFile = true;
+            }
+            else
+            {
+                _saveToFile = false;
             }
-
-            _saveToFile = saveToFile;
         }
 
         /// <summary>
@@ -205,6 +208,7 @@ internal void SetSaveToFile(bool saveToFile)
         /// <param name="saveDirectory">The directory to which logs can be saved</param>
         internal void SetSaveDirectory(string saveDirectory)
         {
+            if (string.IsNullOrEmpty(saveDirectory)) throw new ArgumentNullException(nameof(saveDirectory));
             if (!Directory.Exists(saveDirectory)) throw new IOException("The selected log directory (" + saveDirectory + ") does not exist!");
 
             // Format the directory string

MemPlus/Business/UTILS/Utils.cs

@@ -286,20 +286,21 @@ internal static bool AutoStartUp()
         /// </summary>
         /// <param name="file">The file for which the Process object should be retrieved</param>
         /// <returns></returns>
-        internal static Process GetProcessForFile(string file)
+        internal static List<Process> GetProcessesForFile(string file)
         {
+            List<Process> processes = new List<Process>();
             foreach (Process p in Process.GetProcesses())
             {
                 try
                 {
-                    if (p.MainModule.FileName == file) return p;
+                    if (p.MainModule.FileName == file) processes.Add(p);
                 }
                 catch (Exception)
                 {
                     // ignored
                 }
             }
-            return null;
+            return processes;
         }
     }
 }

MemPlus/Views/Windows/MainWindow.xaml.cs

@@ -1,4 +1,5 @@
 using System;
+using System.Collections.Generic;
 using System.ComponentModel;
 using System.Diagnostics;
 using System.Reflection;
@@ -57,6 +58,15 @@ public MainWindow()
         {
             try
             {
+                if (Properties.Settings.Default.SaveLogsToFile)
+                {
+                    if (string.IsNullOrEmpty(Properties.Settings.Default.LogPath))
+                    {
+                        Properties.Settings.Default.SaveLogsToFile = false;
+                        Properties.Settings.Default.Save();
+                    }
+                }
+                
                 _logController = new LogController
                 (
                     Properties.Settings.Default.LoggingEnabled,
@@ -1079,18 +1089,20 @@ private void MainWindow_Drop(object sender, System.Windows.DragEventArgs e)
             if (!e.Data.GetDataPresent(DataFormats.FileDrop)) return;
             string[] files = (string[])e.Data.GetData(DataFormats.FileDrop);
             if (files == null) return;
-            if (!System.IO.File.Exists(files[0])) return;
 
             foreach (string s in files)
             {
-                Process process = Utils.GetProcessForFile(s);
+                if (!System.IO.File.Exists(s)) continue;
+                List<Process> processes = Utils.GetProcessesForFile(s);
                 try
                 {
-                    if (process == null) continue;
-
-                    _logController.AddLog(new RamLog("Emptying working set for process: " + process.ProcessName));
-                    NativeMethods.EmptyWorkingSet(process.Handle);
-                    _logController.AddLog(new RamLog("Successfully emptied working set for process " + process.ProcessName));
+                    if (processes.Count == 0) return;
+                    foreach (Process p in processes)
+                    {
+                        _logController.AddLog(new RamLog("Emptying working set for process: " + p.ProcessName));
+                        NativeMethods.EmptyWorkingSet(p.Handle);
+                        _logController.AddLog(new RamLog("Successfully emptied working set for process " + p.ProcessName));
+                    }
                 }
                 catch (Exception ex)
                 {

MemPlus/Views/Windows/SettingsWindow.xaml.cs

@@ -345,6 +345,7 @@ private void ResetSettings()
 
                 _logController.SetAutoClear(Properties.Settings.Default.LogClearAuto);
                 _logController.SetAutoClearInterval(Properties.Settings.Default.LogClearInterval);
+                _logController.SetSaveDirectory(Properties.Settings.Default.LogPath);
                 _logController.SetSaveToFile(Properties.Settings.Default.SaveLogsToFile);
 
                 ChangeVisualStyle();