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.

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

cmd 配下のいくつかのファイルを UTF-8 (BOM付) に単純変換 #248

Merged
merged 1 commit into from
Jul 13, 2018

Conversation

kobake
Copy link
Member

@kobake kobake commented Jul 12, 2018

該当フォルダ内の文字コードを変換しても支障のないものだけを対象に変換を行いました。

cd sakura_core/cmd
nkf --overwrite --oc=UTF-8-BOM *.cpp
nkf --overwrite --oc=UTF-8-BOM *.h
git checkout CViewCommander_TagJump.cpp   … ASCII以外の文字定数・文字列定数があるので変換から除外
git checkout CViewCommander_File.cpp      … ASCII以外の文字定数・文字列定数があるので変換から除外
git checkout CViewCommander_Clipboard.cpp … ASCII以外の文字定数・文字列定数があるので変換から除外
git checkout CViewCommander_Grep.cpp      … ASCII以外の文字定数・文字列定数があるので変換から除外

確認方法

WinMerge で変更前と変更後を比較すると、文字コード以外の変更が無いことが確認できます。

関連 Issues

ソースコードのUnicode化 #112

@kobake kobake changed the title cmd 配下で文字エンコーディングを変更しても支障ないファイルのみを UTF-8 (BOM付) に単純変換 cmd 配下のいくつかのファイルを UTF-8 (BOM付) に単純変換 Jul 12, 2018
@berryzplus
Copy link
Contributor

これレビューコメントじゃないんですが、ANSIコードページというのはwindowsプログラム用語で「ローカライズされたコードページ」を意味すると思います。

日本語の場合shiftjisです。

正確にはasciiでは。

cd sakura_core/cmd
nkf --overwrite --oc=UTF-8-BOM *.cpp
nkf --overwrite --oc=UTF-8-BOM *.h
git checkout CViewCommander_TagJump.cpp   … ASCII以外の文字定数・文字列定数があるので変換から除外
git checkout CViewCommander_File.cpp      … ASCII以外の文字定数・文字列定数があるので変換から除外
git checkout CViewCommander_Clipboard.cpp … ASCII以外の文字定数・文字列定数があるので変換から除外
git checkout CViewCommander_Grep.cpp      … ASCII以外の文字定数・文字列定数があるので変換から除外
@kobake
Copy link
Member Author

kobake commented Jul 12, 2018

正確にはasciiでは。

あ、そうでした……。PR本文とコミットメッセージ書き換えますね。

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

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

LGTMです。
チェックアウトして文字コード以外の差異がないことを確認しました。

@kobake kobake merged commit 4f185b7 into sakura-editor:master Jul 13, 2018
@kobake kobake deleted the cmd-utf8 branch July 13, 2018 10:24
@m-tmatma m-tmatma added this to the next release milestone Jul 25, 2018
@m-tmatma m-tmatma added the refactoring リファクタリング 【ChangeLog除外】 label Jul 25, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
cmd 配下のいくつかのファイルを UTF-8 (BOM付) に単純変換
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring リファクタリング 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants