From ff58fa99f41f315e1f03ee330ea7b528e5c65f55 Mon Sep 17 00:00:00 2001
From: David Kocik <kocikdav@gmail.com>
Date: Fri, 13 Dec 2019 11:52:08 +0100
Subject: [PATCH 1/2] comments

---
 src/slic3r/GUI/RemovableDriveManager.cpp | 24 ++++++++++++++++++----
 src/slic3r/GUI/RemovableDriveManager.hpp | 26 ++++++++++++++++--------
 2 files changed, 37 insertions(+), 13 deletions(-)

diff --git a/src/slic3r/GUI/RemovableDriveManager.cpp b/src/slic3r/GUI/RemovableDriveManager.cpp
index cd360b580..02681b7da 100644
--- a/src/slic3r/GUI/RemovableDriveManager.cpp
+++ b/src/slic3r/GUI/RemovableDriveManager.cpp
@@ -30,7 +30,7 @@ INT_PTR WINAPI WinProcCallback(HWND hWnd, UINT message, WPARAM wParam, LPARAM lP
 void RemovableDriveManager::search_for_drives()
 {
 	m_current_drives.clear();
-	m_current_drives.reserve(26);
+	//get logical drives flags by letter in alphabetical order
 	DWORD drives_mask = GetLogicalDrives();
 	for (size_t i = 0; i < 26; i++)
 	{
@@ -39,6 +39,7 @@ void RemovableDriveManager::search_for_drives()
 			std::string path (1,(char)('A' + i));
 			path+=":";
 			UINT drive_type = GetDriveTypeA(path.c_str());
+			// DRIVE_REMOVABLE on W are sd cards and usb thumbnails (not usb harddrives)
 			if (drive_type ==  DRIVE_REMOVABLE)
 			{
 				// get name of drive
@@ -51,10 +52,12 @@ void RemovableDriveManager::search_for_drives()
 				BOOL error = GetVolumeInformationW(wpath.c_str(), &volume_name[0], sizeof(volume_name), NULL, NULL, NULL, &file_system_name[0], sizeof(file_system_name));
 				if(error != 0)
 				{
+					/*
 					if (volume_name == L"")
 					{
 						volume_name = L"REMOVABLE DRIVE";
 					}
+					*/
 					if (file_system_name != L"")
 					{
 						ULARGE_INTEGER free_space;
@@ -78,6 +81,7 @@ void RemovableDriveManager::eject_drive(const std::string &path)
 	{
 		if ((*it).path == path)
 		{
+			// get handle to device
 			std::string mpath = "\\\\.\\" + path;
 			mpath = mpath.substr(0, mpath.size() - 1);
 			HANDLE handle = CreateFileA(mpath.c_str(), GENERIC_READ | GENERIC_WRITE, FILE_SHARE_READ | FILE_SHARE_WRITE, nullptr, OPEN_EXISTING, 0, nullptr);
@@ -87,8 +91,12 @@ void RemovableDriveManager::eject_drive(const std::string &path)
 				return;
 			}
 			DWORD deviceControlRetVal(0);
+			//these 3 commands should eject device safely but they dont, the device does disappear from file explorer but the "device was safely remove" notification doesnt trigger.
+			//sd cards does  trigger WM_DEVICECHANGE messege, usb drives dont
+			
 			DeviceIoControl(handle, FSCTL_LOCK_VOLUME, nullptr, 0, nullptr, 0, &deviceControlRetVal, nullptr);
 			DeviceIoControl(handle, FSCTL_DISMOUNT_VOLUME, nullptr, 0, nullptr, 0, &deviceControlRetVal, nullptr);
+			// some implemenatations also calls IOCTL_STORAGE_MEDIA_REMOVAL here but it returns error to me
 			BOOL error = DeviceIoControl(handle, IOCTL_STORAGE_EJECT_MEDIA, nullptr, 0, nullptr, 0, &deviceControlRetVal, nullptr);
 			if (error == 0)
 			{
@@ -130,11 +138,12 @@ std::string RemovableDriveManager::get_drive_from_path(const std::string& path)
 void RemovableDriveManager::register_window()
 {
 	//creates new unvisible window that is recieving callbacks from system
+	// structure to register 
 	WNDCLASSEX wndClass;
 	wndClass.cbSize = sizeof(WNDCLASSEX);
 	wndClass.style = CS_OWNDC | CS_HREDRAW | CS_VREDRAW;
 	wndClass.hInstance = reinterpret_cast<HINSTANCE>(GetModuleHandle(0));
-	wndClass.lpfnWndProc = reinterpret_cast<WNDPROC>(WinProcCallback);
+	wndClass.lpfnWndProc = reinterpret_cast<WNDPROC>(WinProcCallback);//this is callback
 	wndClass.cbClsExtra = 0;
 	wndClass.cbWndExtra = 0;
 	wndClass.hIcon = LoadIcon(0, IDI_APPLICATION);
@@ -169,6 +178,9 @@ void RemovableDriveManager::register_window()
 
 INT_PTR WINAPI WinProcCallback(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
 {
+	// here we need to catch messeges about device removal
+	// problem is that when ejecting usb (how is it implemented above) there is no messege dispached. Only after physical removal of the device.
+	//uncomment register_window() in init() to register and comment update() in GUI_App.cpp (only for windows!) to stop recieving periodical updates 
 	LRESULT lRet = 1;
 	static HDEVNOTIFY hDeviceNotify;
 
@@ -187,6 +199,7 @@ INT_PTR WINAPI WinProcCallback(HWND hWnd, UINT message, WPARAM wParam, LPARAM lP
 	
 	case WM_DEVICECHANGE:
 	{
+		// here is the important
 		if(wParam == DBT_DEVICEREMOVECOMPLETE)
 		{
 -			RemovableDriveManager::get_instance().update(0, true);
@@ -207,9 +220,9 @@ void RemovableDriveManager::search_for_drives()
 {
     
     m_current_drives.clear();
-    m_current_drives.reserve(26);
     
 #if __APPLE__
+	// if on macos obj-c class will enumerate
 	if(m_rdmmm)
 	{
 		m_rdmmm->list_devices();
@@ -287,6 +300,8 @@ void RemovableDriveManager::search_path(const std::string &path,const std::strin
 }
 void RemovableDriveManager::inspect_file(const std::string &path, const std::string &parent_path)
 {
+	//confirms if the file is removable drive and adds it to vector
+
 	//if not same file system - could be removable drive
 	if(!compare_filesystem_id(path, parent_path))
 	{
@@ -335,7 +350,8 @@ void RemovableDriveManager::eject_drive(const std::string &path)
             	}
             }
             std::cout<<"Ejecting "<<(*it).name<<" from "<< correct_path<<"\n";
-
+// there is no usable command in c++ so terminal command is used instead
+// but neither triggers "succesful safe removal messege"
             std::string command = "";
 #if __APPLE__
             command = "diskutil unmount ";
diff --git a/src/slic3r/GUI/RemovableDriveManager.hpp b/src/slic3r/GUI/RemovableDriveManager.hpp
index b4fc71e26..ac1645df7 100644
--- a/src/slic3r/GUI/RemovableDriveManager.hpp
+++ b/src/slic3r/GUI/RemovableDriveManager.hpp
@@ -29,26 +29,34 @@ public:
 	}
 	RemovableDriveManager(RemovableDriveManager const&) = delete;
 	void operator=(RemovableDriveManager const&) = delete;
-	
-	//update() searches for removable devices, returns false if empty.
+	//call only once. on apple register for unmnount callbacks. on windows register for device notification is prepared but not called (eject usb drive on widnows doesnt trigger the callback, sdc ard does), also enumerates devices for first time so init shoud be called on linux too.
 	void init();
-	bool update(const long time = 0,const bool check = false);  //time = 0 is forced update, time expects wxGetLocalTime()
+	//update() searches for removable devices, returns false if empty. /time = 0 is forced update, time expects wxGetLocalTime()
+	bool update(const long time = 0,const bool check = false);  
 	bool is_drive_mounted(const std::string &path);
 	void eject_drive(const std::string &path);
+	//returns path to last drive which was used, if none was used, returns device that was enumerated last
 	std::string get_last_save_path();
+	//returns path to last drive which was used, if none was used, returns empty string
 	std::string get_drive_path();
 	std::vector<DriveData> get_all_drives();
 	bool is_path_on_removable_drive(const std::string &path);
-	void add_callback(std::function<void()> callback); // callback will notify only if device with last save path was removed
-	void erase_callbacks(); // erases all callbacks added by add_callback()
+	// callback will notify only if device with last save path was removed
+	void add_callback(std::function<void()> callback);
+	// erases all callbacks added by add_callback()
+	void erase_callbacks(); 
+	// marks one of the eveices in vector as last used
 	void set_last_save_path(const std::string &path);
 	bool is_last_drive_removed();
-	bool is_last_drive_removed_with_update(const long time = 0); // param as update()
+	// param as update()
+	bool is_last_drive_removed_with_update(const long time = 0);
 private:
     RemovableDriveManager();
 	void search_for_drives();
+	//triggers callbacks if last used drive was removed
 	void check_and_notify();
-	std::string get_drive_from_path(const std::string& path);//returns drive path (same as path in DriveData) if exists otherwise empty string ""
+	//returns drive path (same as path in DriveData) if exists otherwise empty string ""
+	std::string get_drive_from_path(const std::string& path);
 	void reset_last_save_path();
 
 	std::vector<DriveData> m_current_drives;
@@ -58,8 +66,8 @@ private:
 	std::string m_last_save_path;
 
 #if _WIN32
+	//registers for notifications by creating invisible window
 	void register_window();
-	//INT_PTR WINAPI WinProcCallback(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam);
 #else
 #if __APPLE__
 	RDMMMWrapper * m_rdmmm;
@@ -69,7 +77,7 @@ private:
     void inspect_file(const std::string &path, const std::string &parent_path);
 #endif
 };
-    
+// apple wrapper for RemovableDriveManagerMM which searches for drives and/or ejects them    
 #if __APPLE__
 class RDMMMWrapper
 {

From 8895e944cf2deca3b894415ba05fe281af4a7456 Mon Sep 17 00:00:00 2001
From: David Kocik <kocikdav@gmail.com>
Date: Fri, 13 Dec 2019 13:04:09 +0100
Subject: [PATCH 2/2] comments

---
 src/slic3r/GUI/Plater.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/slic3r/GUI/Plater.cpp b/src/slic3r/GUI/Plater.cpp
index 417afa0a9..0dc09d9d8 100644
--- a/src/slic3r/GUI/Plater.cpp
+++ b/src/slic3r/GUI/Plater.cpp
@@ -3517,7 +3517,7 @@ void Plater::priv::on_process_completed(wxCommandEvent &evt)
         break;
     default: break;
     }
-
+	//added to show disconnect_button after writing
 	show_action_buttons(false);
 
     if (canceled) {