異常停止不具合を修正

Author: katahiromz

Utils.cpp

@@ -640,23 +640,14 @@ XGStringW XgBinToHex(const void *ptr, size_t size)
 // 16進をバイナリにする。
 void XgHexToBin(std::vector<BYTE>& data, const XGStringW& str)
 {
-    WCHAR sz[3];
     data.clear();
-    bool flag = false;
-    sz[2] = 0;
-    for (auto& ch : str)
+    WCHAR sz[3] = {};
+    size_t len = str.size() & ~size_t(1); // 奇数なら最後の1文字を捨てる（既存動作と同じ）
+    for (size_t i = 0; i < len; i += 2)
     {
-        if (flag)
-        {
-            sz[1] = ch;
-            const auto b = static_cast<BYTE>(wcstol(sz, nullptr, 16));
-            data.insert(data.end(), b);
-        }
-        else
-        {
-            sz[0] = ch;
-        }
-        flag = !flag;
+        sz[0] = str[i];
+        sz[1] = str[i + 1];
+        data.push_back(static_cast<BYTE>(wcstol(sz, nullptr, 16)));
     }
 }
 

XG_SettingsDialog.cpp

@@ -398,8 +398,10 @@ BOOL XG_SettingsDialog::DoImportLooks(HWND hwnd, LPCWSTR pszFileName)
         std::vector<BYTE> data;
         XgHexToBin(data, szText);
         XGStringW str;
-        str.resize(data.size() / sizeof(WCHAR));
-        memcpy(&str[0], data.data(), data.size());
+        size_t charCount = data.size() / sizeof(WCHAR);
+        str.resize(charCount);
+        if (charCount)
+            memcpy(&str[0], data.data(), charCount * sizeof(WCHAR)); // バッファ分だけコピー
         ComboBox_RealSetText(hCmb2, str.c_str());
     }
 
@@ -913,7 +915,6 @@ XG_SettingsDialog::DialogProcDx(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lPar
         case psh3: // マスのフォントリセット
             m_lfCellFont = {};
             ::SetDlgItemTextW(hwnd, edt1, L"");
-            PropSheet_Changed(GetParent(hwnd), hwnd);
             break;
 
         case psh4: // 小さいフォントリセット

XgViewSettings.cpp

@@ -239,6 +239,9 @@ XgViewSettingsDlgProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
                     // 表示を更新。
                     XgUpdateImage(xg_hMainWnd);
                     SendMessageW(xg_hMainWnd, WM_SIZE, 0, 0);
+
+                    SetWindowLongPtr(hwnd, DWLP_MSGRESULT, PSNRET_NOERROR);
+                    return TRUE;
                 }
                 break;
             }