Changeset 2004 for trunk/eraser

Timestamp:

5/1/2010 1:02:15 PM (3 years ago)

Author:

lowjoel

Message:

Use response files when the command line may be too long. This allows infinitely large numbers of files to be erased.

Location:

trunk/eraser/ShellExt

Files:

• CtxMenu.cpp (modified) (1 diff)
• ShellExt.rc (modified) (5 diffs)
• resource.h (modified) (1 diff)
• stdafx.h (modified) (1 diff)

trunk/eraser/ShellExt/CtxMenu.cpp

@@ -836 +836 @@
         std::wstring finalParameters;
         {
-            std::wostringstream parametersStrm;
-            parametersStrm << "\"" << action << L"\" /quiet " << parameters;
-            finalParameters = parametersStrm.str();
+            //Depending on the length of the argument, we either use a response file
+            //or pass the arguments directly.
+            if (parameters.length() > 8192)
+            {
+                //The parameters are greater than 8kb, the response file would be
+                //more efficient.
+                wchar_t buffer[MAX_PATH];
+                wchar_t tempPath[MAX_PATH];
+                if (!GetTempPath(sizeof(tempPath) / sizeof(tempPath[0]), tempPath) ||
+                    !GetTempFileName(tempPath, L"ers", 0, buffer))
+                {
+                    throw LoadString(IDS_ERROR_CANNOT_GENERATE_TEMP_FILE);
+                }
+
+                std::wofstream stream(buffer);
+                stream << "\"" << action << L"\" /quiet " << parameters;
+
+                finalParameters = L"\"@";
+                finalParameters += buffer;
+                finalParameters += '"';
+            }
+            else
+            {
+                //Short command line, pass directly to the program
+                std::wostringstream stream;
+                stream << "\"" << action << L"\" /quiet " << parameters;
+                finalParameters = stream.str();
+            }
         }
 

trunk/eraser/ShellExt/ShellExt.rc

@@ -37 +37 @@
 
 /////////////////////////////////////////////////////////////////////////////
-// English (U.S.) resources
+// Polish resources
+
+#if !defined(AFX_RESOURCE_DLL) || defined(AFX_TARG_PLK)
+#ifdef _WIN32
+LANGUAGE LANG_POLISH, SUBLANG_DEFAULT
+#pragma code_page(1250)
+#endif //_WIN32
+
+/////////////////////////////////////////////////////////////////////////////
+//
+// String Table
+//
+
+STRINGTABLE 
+BEGIN
+    IDS_PROJNAME            "EraserShellExt"
+    IDS_ERASER              "Eraser"
+    IDS_ERASERSHELLEXT      "Eraser Shell Extension"
+END
+
+STRINGTABLE 
+BEGIN
+    IDS_ERROR_UNKNOWN       "Wyst¹pi³ nieznany b³¹d."
+    IDS_ERROR_UNKNOWNACTION "Wyst¹pi³o niew³aœciwe ¿¹danie polecenia o ID %d .\n\nEraser nie móg³ przetworzyæ ¿¹dania."
+    IDS_ERROR_CANNOTFINDERASER 
+                            "Aplikacja Eraser nie mog³abyæ uruchomiona. Upewnij siê, ¿e twoja instalacja Erasera nie jest uszkodzona: spróbuj ponownie uruchomiæ kreatora instalacji Eraser i wybierz opcjê napraw instlacje."
+    IDS_ERROR_ACCESSDENIED  "Odmówiono dostêpu do Erasera. To mo¿e byæ z powodu braku uprawnieñ administratora, lub odwo³ano ¿¹danie Uruchom jako administrator."
+    IDS_ERROR_TOO_MANY_FILES 
+                            "Polecenie nie mog³o zrealizowane przez bie¿¹ce wyst¹pienie Erasera, poniewa¿ zbyt wiele plików by³o wybranych do wymazania.\n\nWybierz mniej plików i spróbuj ponownie."
+    IDS_ERROR_MISC          "Wyst¹pi³ b³¹d:\n\n%s"
+END
+
+STRINGTABLE 
+BEGIN
+    IDS_HELPSTRING_ERASE    "Wymazuje wybrane elementy."
+    IDS_HELPSTRING_ERASEONRESTART 
+                            "Wymazuje wybrane elementy przy ponownym uruchomieniu komputera."
+    IDS_HELPSTRING_ERASEUNUSEDSPACE 
+                            "Wymazuje nieu¿ywane miejsce wybranych napêdów."
+    IDS_HELPSTRING_SECUREMOVE 
+                            "Przenieœ bezpiecznie zaznaczone elementy do nowej lokalizacji."
+END
+
+STRINGTABLE 
+BEGIN
+    IDS_ACTION_ERASE        "W&yma¿"
+    IDS_ACTION_ERASERESTART "Wyma¿ przy ponownym u&ruchomieniu"
+    IDS_ACTION_ERASEUNUSEDSPACE "Wyma¿ Nnie&u¿ywane miejsce"
+    IDS_ACTION_SECUREMOVE   "Bezpieczne przeniesienie"
+END
+
+#endif    // Polish resources
+/////////////////////////////////////////////////////////////////////////////
+
+
+/////////////////////////////////////////////////////////////////////////////
+// English (Neutral) resources
 
 #if !defined(AFX_RESOURCE_DLL) || defined(AFX_TARG_ENU)
@@ -119 +175 @@
 
 IDR_ERASERSHELLEXT      REGISTRY                "ShellExt.rgs"
-/////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
@@ -143 +198 @@
                             "The command could not be sent to the running Eraser instance because too many files were selected for erasure.\n\nSelect fewer files and try again."
     IDS_ERROR_MISC          "An error has occurred:\n\n%s"
+    IDS_ERROR_CANNOT_GENERATE_TEMP_FILE 
+                            "Eraser could not generate a temporary file to store the list of files selected for erasure."
 END
 
@@ -164 +221 @@
 END
 
-#endif    // English (U.S.) resources
-
-
-/////////////////////////////////////////////////////////////////////////////
-// Italian (Italy) resources
+#endif    // English (Neutral) resources
+/////////////////////////////////////////////////////////////////////////////
+
+
+/////////////////////////////////////////////////////////////////////////////
+// Italian (Neutral) resources
 
 #if !defined(AFX_RESOURCE_DLL) || defined(AFX_TARG_ITA)
@@ -219 +277 @@
 END
 
-#endif    // Italian (Italy) resources
-/////////////////////////////////////////////////////////////////////////////
-
-/////////////////////////////////////////////////////////////////////////////
-// Polish (Poland) resources
-
-#if !defined(AFX_RESOURCE_DLL) || defined(AFX_TARG_POL)
-#ifdef _WIN32
-LANGUAGE LANG_POLISH, SUBLANG_NEUTRAL
-#pragma code_page(1250)
-#endif //_WIN32
-
-/////////////////////////////////////////////////////////////////////////////
-//
-// String Table
-//
-
-STRINGTABLE 
-BEGIN
-    IDS_PROJNAME            "EraserShellExt"
-    IDS_ERASER              "Eraser"
-    IDS_ERASERSHELLEXT      "Eraser Shell Extension"
-END
-
-STRINGTABLE 
-BEGIN
-    IDS_ERROR_UNKNOWN       "Wyst¹pi³ nieznany b³¹d."
-    IDS_ERROR_UNKNOWNACTION "Wyst¹pi³o niew³aœciwe ¿¹danie polecenia o ID %d .\n\nEraser nie móg³ przetworzyæ ¿¹dania."
-    IDS_ERROR_CANNOTFINDERASER 
-                            "Aplikacja Eraser nie mog³abyæ uruchomiona. Upewnij siê, ¿e twoja instalacja Erasera nie jest uszkodzona: spróbuj ponownie uruchomiæ kreatora instalacji Eraser i wybierz opcjê napraw instlacje."
-    IDS_ERROR_ACCESSDENIED  "Odmówiono dostêpu do Erasera. To mo¿e byæ z powodu braku uprawnieñ administratora, lub odwo³ano ¿¹danie Uruchom jako administrator."
-    IDS_ERROR_TOO_MANY_FILES 
-                            "Polecenie nie mog³o zrealizowane przez bie¿¹ce wyst¹pienie Erasera, poniewa¿ zbyt wiele plików by³o wybranych do wymazania.\n\nWybierz mniej plików i spróbuj ponownie."
-    IDS_ERROR_MISC          "Wyst¹pi³ b³¹d:\n\n%s"
-END
-
-STRINGTABLE 
-BEGIN
-    IDS_HELPSTRING_ERASE    "Wymazuje wybrane elementy."
-    IDS_HELPSTRING_ERASEONRESTART 
-                            "Wymazuje wybrane elementy przy ponownym uruchomieniu komputera."
-    IDS_HELPSTRING_ERASEUNUSEDSPACE 
-                            "Wymazuje nieu¿ywane miejsce wybranych napêdów."
-    IDS_HELPSTRING_SECUREMOVE 
-                            "Przenieœ bezpiecznie zaznaczone elementy do nowej lokalizacji."
-END
-
-STRINGTABLE 
-BEGIN
-    IDS_ACTION_ERASE        "W&yma¿"
-    IDS_ACTION_ERASERESTART "Wyma¿ przy ponownym u&ruchomieniu"
-    IDS_ACTION_ERASEUNUSEDSPACE "Wyma¿ Nnie&u¿ywane miejsce"
-    IDS_ACTION_SECUREMOVE   "Bezpieczne przeniesienie"
-END
-
-#endif    // Polish (Poland) resources
+#endif    // Italian (Neutral) resources
 /////////////////////////////////////////////////////////////////////////////
 

trunk/eraser/ShellExt/resource.h

@@ -21 +21 @@
 #define IDS_ERROR_TOO_MANY_FILES        404
 #define IDS_ERROR_MISC                  405
+#define IDS_ERROR_CANNOT_GENERATE_TEMP_FILE 406
 
 // Next default values for new objects

trunk/eraser/ShellExt/stdafx.h

@@ -51 +51 @@
 #include <string>
 #include <sstream>
+#include <fstream>
 
 #undef max