Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 23 commits into from
Oct 15, 2018
Merged

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

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
395c925
_WIN32_WINNTを_WIN32_WINNT_WIN7に更新
berryzplus Oct 13, 2018
6667aba
vista以降チェックの方法を他に合わせる
berryzplus Oct 13, 2018
ccb4911
実効性がないバージョンチェックを除去
berryzplus Oct 13, 2018
b0fd56c
バージョン確認関数のリファクタリング
berryzplus Oct 13, 2018
8409397
SDKの定数を独自定義するのをやめる
berryzplus Oct 13, 2018
abf3ab1
IsWin32NT除去
berryzplus Oct 13, 2018
3707b82
HasWinHelpContentsProblem除去
berryzplus Oct 13, 2018
9c013c8
OsSupportReconvertを除去
berryzplus Oct 13, 2018
1beb197
IsWinV5forOfnを除去
berryzplus Oct 13, 2018
53274ea
IsWinVista_or_laterを除去
berryzplus Oct 13, 2018
e14801a
IsWinXP_or_later除去
berryzplus Oct 13, 2018
0ba38d4
IsWin2000_or_laterを除去
berryzplus Oct 13, 2018
b726762
IsWinMeを除去
berryzplus Oct 13, 2018
17db5f6
IsWineを除去
berryzplus Oct 13, 2018
46a149a
COsVersionInfo.hのインクルードを除去
berryzplus Oct 13, 2018
66bffd0
不要となったファイルを除去
berryzplus Oct 13, 2018
1b81d46
要らなくなったGetProcAddressを除去
berryzplus Oct 13, 2018
24d519f
実効性のないコードを除去
berryzplus Oct 13, 2018
05f6df4
CHtmlHelpを他実装に合わせる
berryzplus Oct 13, 2018
1094464
古いOS向けのコメント除去
berryzplus Oct 14, 2018
185e68e
固定配列にstd::vectorを使っていたのをやめる
berryzplus Oct 14, 2018
3e54587
std::wstringをstd::unique_ptrに変える
berryzplus Oct 14, 2018
8ca7819
std::unique_ptrを固定配列に変える
berryzplus Oct 14, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
IsWinVista_or_laterを除去
  • Loading branch information
@berryzplus
berryzplus committed Oct 13, 2018
commit 53274ea2d588005cb8724221cf15587947d8a33d
16 changes: 1 addition & 15 deletions sakura_core/CSelectLang.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -403,21 +403,7 @@ HINSTANCE CSelectLang::ChangeLang( UINT nIndex )
m_psLangInfo = psLangInfo;

// ロケールを設定
// SetThreadUILanguageの呼び出しを試みる
bool isSuccess = false;
if( IsWinVista_or_later() ) {
HMODULE hDll = LoadLibrary( _T("kernel32") );
if ( hDll ) {
typedef short (CALLBACK* SetThreadUILanguageType)(LANGID);
SetThreadUILanguageType _SetThreadUILanguage = (SetThreadUILanguageType)
GetProcAddress(hDll, "SetThreadUILanguage");
isSuccess = _SetThreadUILanguage && _SetThreadUILanguage( m_psLangInfo->wLangId );
FreeLibrary( hDll );
}
}
if ( !isSuccess ) {
SetThreadLocale(MAKELCID( m_psLangInfo->wLangId, SORT_DEFAULT ));
}
::SetThreadUILanguage( m_psLangInfo->wLangId );

return m_psLangInfo->hInstance;
}
Expand Down
2 changes: 0 additions & 2 deletions sakura_core/_os/COsVersionInfo.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,6 @@
#define _COSVERSIONINFO_H_


// 稼働中のOSはvista以降か？＝常に真。
inline bool IsWinVista_or_later() { return true; }
// 稼働中のOSはxp以降か？＝常に真。
inline bool IsWinXP_or_later() { return true; }
// 稼働中のOSはw2k以降か？＝常に真。
Expand Down
10 changes: 5 additions & 5 deletions sakura_core/apiwrap/StdApi.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -276,14 +276,14 @@ namespace ApiWrap{
参考：http://forums.microsoft.com/MSDN-JA/ShowPost.aspx?PostID=3228018&SiteID=7
> Vista で Aero を OFF にすると SetPixel がうまく動かないそうです。
> しかも、SP1 でも修正されていないとか。

一旦はvista以降向けの「不具合」対策をそのまま残します。
vista前後でGDIの考え方が変わってるので、デバッグのやり方を考え直すべきと思います。
by berryzplus 2018/10/13記す。
*/
void SetPixelSurely(HDC hdc,int x,int y,COLORREF c)
{
if (!IsWinVista_or_later()) {
//Vistaより前：SetPixel直呼び出し
::SetPixel(hdc,x,y,c);
}
else {
{
//Vista以降：SetPixelエミュレート
static HPEN hPen = NULL;
static COLORREF clrPen = 0;
Expand Down
4 changes: 2 additions & 2 deletions sakura_core/uiparts/CMenuDrawer.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -844,7 +844,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 ){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

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

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

nFlagAdd = MF_OWNERDRAW;
}
/* 機能のビットマップの情報を覚えておく */
Expand All @@ -855,7 +855,7 @@ void CMenuDrawer::MyAppendMenu(
#ifdef DRAW_MENU_ICON_BACKGROUND_3DFACE
// セパレータかサブメニュー
if( nFlag & (MF_SEPARATOR | MF_POPUP) ){
if( m_pShareData->m_Common.m_sWindow.m_bMenuIcon || !IsWinVista_or_later() ){
if( m_pShareData->m_Common.m_sWindow.m_bMenuIcon ){
nFlagAdd = MF_OWNERDRAW;
}
}
Expand Down
15 changes: 1 addition & 14 deletions sakura_core/window/CEditWnd.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -493,19 +493,6 @@ void CEditWnd::_AdjustInMonitor(const STabGroupInfo& sTabGroupInfo)
::SetWindowPos( GetHwnd(), (dwExStyle & WS_EX_TOPMOST)? HWND_TOPMOST: HWND_NOTOPMOST, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE | SWP_NOACTIVATE );

//タブウインドウ時は現状を維持
/* ウィンドウサイズ継承 */
// Vista 以降の初回表示アニメーション効果を抑止する
if( !IsWinVista_or_later() ){
if( sTabGroupInfo.wpTop.showCmd == SW_SHOWMAXIMIZED )
{
::ShowWindow( GetHwnd(), SW_SHOWMAXIMIZED );
}
else
{
::ShowWindow( GetHwnd(), SW_SHOW );
}
}
else
{
// 初回表示のアニメーション効果を抑止する

Expand Down Expand Up @@ -743,7 +730,7 @@ HWND CEditWnd::Create(
m_bIsActiveApp = ( ::GetActiveWindow() == GetHwnd() ); // 2007.03.08 ryoji

// エディタ－トレイ間でのUI特権分離の確認（Vista UIPI機能） 2007.06.07 ryoji
if( IsWinVista_or_later() ){
{
m_bUIPI = FALSE;
::SendMessage( m_pShareData->m_sHandles.m_hwndTray, MYWM_UIPI_CHECK, (WPARAM)0, (LPARAM)GetHwnd() );
if( !m_bUIPI ){ // 返事が返らない
Expand Down