From 7893b7e67ce4c7bb176e7f9c46a01bd310db76d1 Mon Sep 17 00:00:00 2001 From: kichik Date: Mon, 29 Dec 2003 15:52:42 +0000 Subject: [PATCH] - best compressor wasn't choosing the best compressed file - pressing cancel on the compressor dialog now cancels the compilation git-svn-id: https://svn.code.sf.net/p/nsis/code/NSIS/trunk@3331 212acab6-be3b-0410-9dea-997c60f758d6 --- Contrib/Makensisw/makensisw.cpp | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/Contrib/Makensisw/makensisw.cpp b/Contrib/Makensisw/makensisw.cpp index d3ef8b84..f7473aa7 100644 --- a/Contrib/Makensisw/makensisw.cpp +++ b/Contrib/Makensisw/makensisw.cpp @@ -159,7 +159,10 @@ BOOL CALLBACK DialogProc(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam) { SetCompressor(g_sdata.default_compressor); } if(chooseCompressor) { - DialogBox(g_sdata.hInstance,MAKEINTRESOURCE(DLG_COMPRESSOR),g_sdata.hwnd,(DLGPROC)CompressorProc); + if (DialogBox(g_sdata.hInstance,MAKEINTRESOURCE(DLG_COMPRESSOR),g_sdata.hwnd,(DLGPROC)CompressorProc)) { + EnableItems(g_sdata.hwnd); + return TRUE; + } } CompileNSISScript(); return TRUE; @@ -251,7 +254,7 @@ BOOL CALLBACK DialogProc(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam) { } if(g_sdata.compressor == COMPRESSOR_BEST) { if (g_sdata.retcode==0 && FileExists(g_sdata.output_exe)) { - char temp_file_name[MAX_PATH]; + char temp_file_name[1024]; wsprintf(temp_file_name,"%s_makensisw_temp",g_sdata.output_exe); if(!lstrcmpi(g_sdata.compressor_name,compressor_names[(int)COMPRESSOR_SCRIPT+1])) { SetCompressorStats(); @@ -296,7 +299,7 @@ BOOL CALLBACK DialogProc(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam) { if(thisSize != INVALID_FILE_SIZE) { if(prevSize > thisSize) { - CopyFile(temp_file_name,g_sdata.output_exe,false); + CopyFile(g_sdata.output_exe,temp_file_name,false); SetCompressorStats(); g_sdata.best_compressor_name = g_sdata.compressor_name; } @@ -962,12 +965,12 @@ BOOL CALLBACK CompressorProc(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lPara SetCompressor(g_sdata.default_compressor); } - EndDialog(hwndDlg, TRUE); + EndDialog(hwndDlg, 0); break; } case IDCANCEL: { - EndDialog(hwndDlg, TRUE); + EndDialog(hwndDlg, 1); break; } }