ターゲットwindowsをwindows7に上げる #548

berryzplus · 2018-10-13T15:28:26Z

ターゲットwindowsのバージョンをwin2000からwin7に上げます。

副産物として以下の変更を同時に行います。

古いSDKによるビルドで必要だった windows SDK 定数の独自定義の除去
windows vistaまでのosバージョン確認に使用していた独自クラス(COsVersionInfo)の除去
GetProcAddressしなくて良くなった関数呼出を標準の記載に戻す
HtmlHelp関数(API)を呼び出すためのクラスの処理が他実装と乖離していたのを合わせる

この変更の統合により、修正が必要と判断したコードをよく調べてみたらデッドコードだった、という悲劇に遭遇する確立を少しだけ下げることが出来ます。

プロジェクト設定が widows 2000 向けになっていたのを win7 向けに更新します。

ここだけ COsVersionInfoクラスを使っていたのでグローバル関数を呼ぶように修正します。

処理を簡単に記述するために暗黙動的リンクを採用するので、動作対象外OSではそもそも起動できなくなったため、チェックをする意味がなくなりました。シグニチャ訂正をすると他に影響が出るので、一旦内部を空っぽにする対処を行います。

実際に利用されている関数を残して未使用処理を除去。

OPENFILENAMEZの廃止により、利用していた箇所は除去済み。

未使用関数のため影響なし。

IsWine関数は実際には「プロセス毎にキーボードアクセラレータを作る必要があるか？」の判定に使われていた。アクセラレータはプロセス毎に管理するものなので共有しないほうが正しいように思われる。判定文を除去し、常に自プロセスのアクセラレータテーブルを作成するように変更する。

対象OSをwin7に上げたことで不要になったGetProcAddressを除去します。

RAIIクラスを使ったパス移動は、無名スコープを抜ける際に解除されるので意味がありません。カレントディレクトリに変更は比較的重たいので無駄と分かって残しておく意味はないと思われます。

他のCDllImp派生クラスと実装が異なっていた。 HtmlHelpはwindows SDKに定義がある特殊な外部関数なので、 windows SDKの定義とCDllImp実装の両方の特性を活かせるように実装し直します。

sakura_core/print/CPrintPreview.cpp

sakura_core/window/CEditWnd.cpp

単にチェックの有無で倍率設定を切り替える処理となっている。

Dx配列の導出用にstd::vectorを使っていたが、ここの処理では文字列長が固定なので動的サイズ変更しなくてよかった。

sakura_core/window/CEditWnd.cpp

グリフインデックスの受取バッファにstd::wstringを使っていたが、stringの機能は全く使わないので単なるバッファに変更する。

確保量が毎回同じで、かつ、小さいのでスタックに積んで動的確保のコストを削る。

beru · 2018-10-14T16:57:20Z

sakura_core/_main/CProcessFactory.cpp

-	}
-#endif
+	// Windowsバージョンは廃止。
+	// 動作可能バージョン(=windows7以降)でなければ起動できない。


黒翼猫さんが Windows 2000 で無理やり動作させた時は無力感を味わいながら諦めましょう…。

大変お世話になってた気がするので、もし必要なら対応版ブランチを作る感じです。

きっと彼らは最新版追わないので影響ないんじゃないかと思ってます。

beru · 2018-10-14T17:04:08Z

sakura_core/uiparts/CMenuDrawer.cpp

@@ -844,7 +843,7 @@ void CMenuDrawer::MyAppendMenu(
 			// メニュー項目をオーナー描画にして、アイコンを表示する
 			// 2010.03.29 アクセスキーの分を詰めるためいつもオーナードローにする。ただしVista未満限定
 			// Vista以上ではメニューもテーマが適用されるので、オーナードローにすると見た目がXP風になってしまう。
-			if( m_pShareData->m_Common.m_sWindow.m_bMenuIcon || !IsWinVista_or_later() ){
+			if( m_pShareData->m_Common.m_sWindow.m_bMenuIcon ){


PR の変更内容への指摘では無くて余談ですが、もうメニューをオーナードローでやらなくても良いかも知れないですね。いやまぁ確認して Issue 立てろと言われたらそれまでですが…。

メニューのオーナードローをやめる対応は後回しにしました。

これマージした後に ‘#if’ で無効化されてるコードを除去するのをやろうとしていて、それの後くらいがいいかなと思っています。

メニュー処理はコピペで二重化されてるので、整理して一本化するのが先かもです。

beru

メニューバーにメッセージを表示する処理の変更内容が他と比べてちょっと異質な感じはありますが問題無いと思います。

確認していませんが、エディタ側の Unicode 文字列描画処理は等幅専用だったりして使えないのかもしれないですね。それか再利用可能な形になっていないとか。

m-tmatma · 2018-10-14T22:21:59Z

ターゲットwindowsのバージョンをwin2000からwin7に上げます。

refactoring のタグは違うように思います。どちらかというと仕様変更だと思います。
気持ちは refactoring なのはわかります。

berryzplus · 2018-10-15T00:01:16Z

確認ありがとうございます。
マージします。

berryzplus · 2018-10-15T00:04:27Z

refactoring のタグは違うように思います。どちらかというと仕様変更だと思います。
気持ちは refactoring なのはわかります。

あとでタグ外しときます。

ds14050 · 2018-10-15T13:00:36Z

Windows Vista での起動確認を報告します。すでにビルドはできませんが動作テストはまだできるようです。

berryzplus · 2018-10-15T14:48:54Z

Windows Vista での起動確認を報告します。
すでにビルドはできませんが動作テストはまだできるようです。

いまのところ、vistaで使えなくなるような変更は入ってないので、起動はできる想定でした。
当面は vista は「意識しない」というだけで「使えるはず」な状態が続くと思ってます。

ds14050 · 2018-10-17T07:19:37Z

コミット 05f6df4 からだと思いますが、MinGW ビルドがこけています。

extmodule/CHtmlHelp.cpp: In member function 'virtual bool CHtmlHelp::InitDllImp()':
extmodule/CHtmlHelp.cpp:53:5: error: invalid conversion from 'CHtmlHelp::FnPtr_HtmlHelp' {aka 'HWND__* (*)(HWND__*, const wchar_t*, unsigned int, long long unsigned int)'} to 'void*' [-fpermissive]
   { m_pfnHtmlHelp,  "HtmlHelpW" },
     ^~~~~~~~~~~~~
mingw32-make: *** [Makefile:448: extmodule/CHtmlHelp.o] Error 1

berryzplus · 2018-10-17T15:08:09Z

MinGW ビルドがこけています。

こういうのがあるから appveyor に組み込んでおきたかったんです（キリッ

・・・調べて対応します 😭

…target_winver ターゲットwindowsをwindows7に上げる

berryzplus added 6 commits October 13, 2018 23:19

_WIN32_WINNTを_WIN32_WINNT_WIN7に更新

395c925

プロジェクト設定が widows 2000 向けになっていたのを win7 向けに更新します。

vista以降チェックの方法を他に合わせる

6667aba

ここだけ COsVersionInfoクラスを使っていたのでグローバル関数を呼ぶように修正します。

バージョン確認関数のリファクタリング

b0fd56c

実際に利用されている関数を残して未使用処理を除去。

SDKの定数を独自定義するのをやめる

8409397

IsWin32NT除去

abf3ab1

berryzplus force-pushed the feature/upgrade_target_winver branch from bdaaae5 to 05f6df4 Compare October 13, 2018 15:39

berryzplus added 13 commits October 14, 2018 00:39

HasWinHelpContentsProblem除去

3707b82

OsSupportReconvertを除去

9c013c8

IsWinV5forOfnを除去

1beb197

OPENFILENAMEZの廃止により、利用していた箇所は除去済み。

IsWinVista_or_laterを除去

53274ea

IsWinXP_or_later除去

e14801a

IsWin2000_or_laterを除去

0ba38d4

IsWinMeを除去

b726762

未使用関数のため影響なし。

COsVersionInfo.hのインクルードを除去

46a149a

不要となったファイルを除去

66bffd0

要らなくなったGetProcAddressを除去

1b81d46

対象OSをwin7に上げたことで不要になったGetProcAddressを除去します。

CHtmlHelpを他実装に合わせる

05f6df4

他のCDllImp派生クラスと実装が異なっていた。 HtmlHelpはwindows SDKに定義がある特殊な外部関数なので、 windows SDKの定義とCDllImp実装の両方の特性を活かせるように実装し直します。

beru reviewed Oct 13, 2018

View reviewed changes

sakura_core/print/CPrintPreview.cpp Show resolved Hide resolved

beru reviewed Oct 13, 2018

View reviewed changes

sakura_core/window/CEditWnd.cpp Outdated Show resolved Hide resolved

berryzplus added 2 commits October 14, 2018 13:14

古いOS向けのコメント除去

1094464

単にチェックの有無で倍率設定を切り替える処理となっている。

固定配列にstd::vectorを使っていたのをやめる

185e68e

Dx配列の導出用にstd::vectorを使っていたが、ここの処理では文字列長が固定なので動的サイズ変更しなくてよかった。

berryzplus added enhancement ■機能追加 refactoring リファクタリング【ChangeLog除外】 labels Oct 14, 2018

berryzplus added this to the next release milestone Oct 14, 2018

beru reviewed Oct 14, 2018

View reviewed changes

sakura_core/window/CEditWnd.cpp Outdated Show resolved Hide resolved

berryzplus added 2 commits October 14, 2018 19:10

std::wstringをstd::unique_ptrに変える

3e54587

グリフインデックスの受取バッファにstd::wstringを使っていたが、stringの機能は全く使わないので単なるバッファに変更する。

std::unique_ptrを固定配列に変える

8ca7819

確保量が毎回同じで、かつ、小さいのでスタックに積んで動的確保のコストを削る。

beru reviewed Oct 14, 2018

View reviewed changes

beru approved these changes Oct 14, 2018

View reviewed changes

berryzplus merged commit 51d5907 into sakura-editor:master Oct 15, 2018

beru removed the refactoring リファクタリング【ChangeLog除外】 label Oct 15, 2018

berryzplus deleted the feature/upgrade_target_winver branch October 15, 2018 14:48

berryzplus mentioned this pull request Oct 18, 2018

MinGWビルドのエラーに対処する #559

Merged

m-tmatma mentioned this pull request May 4, 2019

Inno Setup での OS の対応バージョンを Windows 7 以降に変更 #895

Merged

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019

Merge pull request sakura-editor#548 from berryzplus/feature/upgrade_…

6522308

…target_winver ターゲットwindowsをwindows7に上げる

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ターゲットwindowsをwindows7に上げる #548

ターゲットwindowsをwindows7に上げる #548

berryzplus commented Oct 13, 2018

beru Oct 14, 2018

berryzplus Oct 14, 2018

beru Oct 14, 2018

berryzplus Oct 14, 2018

beru left a comment

m-tmatma commented Oct 14, 2018

berryzplus commented Oct 15, 2018

berryzplus commented Oct 15, 2018

ds14050 commented Oct 15, 2018

berryzplus commented Oct 15, 2018

ds14050 commented Oct 17, 2018

berryzplus commented Oct 17, 2018

ターゲットwindowsをwindows7に上げる #548

ターゲットwindowsをwindows7に上げる #548

Conversation

berryzplus commented Oct 13, 2018

beru Oct 14, 2018

Choose a reason for hiding this comment

berryzplus Oct 14, 2018

Choose a reason for hiding this comment

beru Oct 14, 2018

Choose a reason for hiding this comment

berryzplus Oct 14, 2018

Choose a reason for hiding this comment

beru left a comment

Choose a reason for hiding this comment

m-tmatma commented Oct 14, 2018

berryzplus commented Oct 15, 2018

berryzplus commented Oct 15, 2018

ds14050 commented Oct 15, 2018

berryzplus commented Oct 15, 2018

ds14050 commented Oct 17, 2018

berryzplus commented Oct 17, 2018