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

Jump to bottom

新規インストール時に言語設定に英語を適用する処理方法の変更 #749

Merged
merged 2 commits into from
Jan 5, 2019
Merged

新規インストール時に言語設定に英語を適用する処理方法の変更 #749

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a507336
新規インストール時に言語設定に英語を適用する処理方法の変更
beru Jan 4, 2019
3b0f5e0
レビュー指摘対応：マジックナンバーを書かずにマクロを使うように変更
beru Jan 5, 2019
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
6 changes: 0 additions & 6 deletions installer/sakura-common.iss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,8 +111,6 @@ en.ReadyMemo_ExecProfileDir=Same as the executable file
ja.ReadyMemo_ExecProfileDir=実行ファイルと同一ディレクトリ
en.ReadyMemo_VirtualStoreEnable=Enable
ja.ReadyMemo_VirtualStoreEnable=有効
en.languageDLL=sakura_lang_en_US.dll
ja.languageDLL=



Expand Down Expand Up @@ -249,10 +247,6 @@ FileName: "{app}\sakura.exe"; Description: "{cm:StartNow}"; WorkingDir: "{app}";
[Dirs]
Name: "{userappdata}\sakura"; Components: main; Tasks: startmenu; Check: isMultiUserEnabled

[Ini]
Filename: "{userappdata}\sakura\sakura.ini"; Section: "Common"; Key: "szLanguageDll"; String: "{cm:languageDLL}"; Check: isMultiUserEnabled
Filename: "{app}\sakura.ini"; Section: "Common"; Key: "szLanguageDll"; String: "{cm:languageDLL}"; Check: isMultiUserDisabled

[Code]
var
MultiUserPage: TInputOptionWizardPage;
Expand Down
18 changes: 9 additions & 9 deletions sakura_core/env/CShareData_IO.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@ void CShareData_IO::SaveShareData()

@param[in] bRead true: 読み込み / false: 書き込み
@return 設定データの読み込み/保存が成功したかどうか
@note 読み込みの場合、言語設定切り替え後にMRUエントリが無い場合は新規インストール後とみなし false を返す事で初期設定を適用させる

@date 2004-01-11 D.S.Koba CProfile変更によるコード簡略化
@date 2005-04-05 D.S.Koba 各セクションの入出力を関数として分離
Expand Down Expand Up @@ -96,6 +95,15 @@ bool CShareData_IO::ShareData_IO_2( bool bRead )
if( bRead ){
if( !cProfile.ReadProfile( szIniFileName ) ){
/* 設定ファイルが存在しない */
LANGID langId = GetUserDefaultUILanguage();
// Windowsの表示言語が日本語でない場合は言語設定を英語にする
if (langId != 0x0411) {
Copy link
Contributor

Choose a reason for hiding this comment

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

wLangId の値はWindows SDKのMAKELANGIDマクロを使って書くものと思っています。

MAKELANGID( LANG_JAPANESE, SUBLANG_JAPANESE_JAPAN )

ただし、このマクロが他で使われているのを見たことはほとんどないっす 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

マジックナンバー埋め込むよりマクロ使うべきですね、指摘ありがとうございます。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

3b0f5e0 で対処しました。

Copy link
Contributor

Choose a reason for hiding this comment

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

他言語DLLの対応状況からして、
「not equal 日本語」ではなく
「equal 英語」に限定するほうがよいような気がします。

つまり langId == 0x409 ということです。

英語(0x409)は以下のように指定します。
MAKELANGID( LANG_ENGLISH, SUBLANG_ENGLISH_US )

Copy link
Contributor Author

Choose a reason for hiding this comment

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

その方が判定としては素直だとは思います。しかし現状言語として設定出来るのが日本語と英語だけなので、日本語使わない人には英語にしとこう、という考え方です。

英語より日本語の方が国際共通語だろJK と真顔で言われたら、ｱｯﾊｲと反応します。

Copy link
Contributor

Choose a reason for hiding this comment

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

L"sakura_lang_en_US.dll" の ファイル存在チェックも条件に加えるのがベターだと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

それも考えましたが有る事が前提にしても良いのではないかと考えて記述しませんでした。
まぁ白状すると GetModuleFileName してから _tsplitpath して PathAppend する手間を省いたんです。

DLLSHAREDATA* pShareData = &GetDllShareData();
_tcscpy(pShareData->m_Common.m_sWindow.m_szLanguageDll, L"sakura_lang_en_US.dll");
cProfile.IOProfileData( L"Common", L"szLanguageDll", MakeStringBufferT( pShareData->m_Common.m_sWindow.m_szLanguageDll ) );
CSelectLang::ChangeLang( pShareData->m_Common.m_sWindow.m_szLanguageDll );
pcShare->RefreshString();
}
return false;
}

Expand Down Expand Up @@ -126,14 +134,6 @@ bool CShareData_IO::ShareData_IO_2( bool bRead )
cProfile.IOProfileData( L"Common", L"szLanguageDll", MakeStringBufferT( pShareData->m_Common.m_sWindow.m_szLanguageDll ) );
CSelectLang::ChangeLang( pShareData->m_Common.m_sWindow.m_szLanguageDll );
pcShare->RefreshString();

// 新規インストール後の設定ファイルは言語設定しか存在しない
// MRUのエントリが無い場合は新規と判断
int _MRU_Counts = 0;
if (!cProfile.IOProfileData( LTEXT("MRU"), LTEXT("_MRU_Counts"), _MRU_Counts )){
// 言語設定の切り替えだけして false を返す事で初期設定を適用させる
return false;
}
}

// Feb. 12, 2006 D.S.Koba
Expand Down