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

ソースコードの文字コードをUTF-8にした後の様々な微修正 #185

Merged
merged 10 commits into from
Jan 16, 2022
Merged

ソースコードの文字コードをUTF-8にした後の様々な微修正 #185

merged 10 commits into from
Jan 16, 2022

Conversation

meltingrabbit
Copy link
Collaborator

@meltingrabbit meltingrabbit commented Jan 16, 2022 •
edited
Loading

概要

ソースコードの文字コードをUTF-8にした後の様々な微修正

Issue

詳細

  • ソースコード類の文字コードをSJISからUTF-8にする #184 の続き
    • コード規約
    • doxygen
    • バッチファイル
    • この際， sils_define.h 周りも整理してしまう
    • エンコーディングチェックCIスクリプトの追加修正
    • コード規約チェックCIスクリプトの追加修正
    • tlm cmd code generatorの修正チェック
    • cmake関連の修正
  • 様々なツール類の修正を行う

関連ToolsのPR

検証結果

  • CIと既存のテストが通ること
  • 関連Toolsを動かして問題ないこと

影響範囲

Tools類にも影響．S2E側にも影響がある可能性があるので，関係各位に連絡する

補足

@meltingrabbit meltingrabbit added documentation Improvements or additions to documentation enhancement New feature or request priority::high priorityg high tools S2E labels Jan 16, 2022
@meltingrabbit meltingrabbit mentioned this pull request Jan 16, 2022
Merged
@meltingrabbit meltingrabbit changed the title ソースコードの文字コードをUTF-8にした後の様々な微修正 WIP: ソースコードの文字コードをUTF-8にした後の様々な微修正 Jan 16, 2022
@meltingrabbit meltingrabbit force-pushed the feature/fix_after_change_code_encoding branch from c9b5c89 to 03a83b4 Compare January 16, 2022 06:31
@meltingrabbit
Copy link
Collaborator Author

S2E (Visual Studio) ビルドでの調整については，

#184 (comment)

を引き継いでこっちでやる．

@sksat
Copy link
Collaborator

sksat commented Jan 16, 2022

このPR，一旦base branchをfeature/change_encode_from_SJIS_to_UTF-8にした方がdiffが見やすそう

@meltingrabbit meltingrabbit changed the base branch from develop to feature/change_encode_from_SJIS_to_UTF-8 January 16, 2022 06:48
@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Jan 16, 2022 •
edited
Loading

した．後で直せばいいか．これ， feature/change_encode_from_SJIS_to_UTF-8 がマージされたらどうなるんだ？（closeされちゃう？）

んで，protect branchじゃなくなるから，マージボタン押せちゃうな．

@sksat
Copy link
Collaborator

sksat commented Jan 16, 2022

自動でブランチ消されるわけですしちょっと怖い気もする(どうなるんだろ)．マージボタン押せちゃうのたしかに...

@meltingrabbit
Copy link
Collaborator Author

deactivate_sils_define.bat に冪等性がないことに気づいてしまった（どんどん // が足されてく）けど，まあいいか．．．普通使わないし

sedかperl環境仮定したい．．．

This was referenced Jan 16, 2022
Merged
Merged
@meltingrabbit meltingrabbit changed the title WIP: ソースコードの文字コードをUTF-8にした後の様々な微修正 ソースコードの文字コードをUTF-8にした後の様々な微修正 Jan 16, 2022
@meltingrabbit
Copy link
Collaborator Author

これで行けるはず．あとは全テスト通し直す．

@yngyu その他誰でも review please~~

@meltingrabbit
Copy link
Collaborator Author

テスト全部通りました．

@meltingrabbit meltingrabbit force-pushed the feature/fix_after_change_code_encoding branch from 6d61e33 to c6bed8e Compare January 16, 2022 13:57
@meltingrabbit meltingrabbit changed the base branch from feature/change_encode_from_SJIS_to_UTF-8 to develop January 16, 2022 13:58
@meltingrabbit
Copy link
Collaborator Author

#184 がマージされたのでrebaseします．

@meltingrabbit meltingrabbit force-pushed the feature/fix_after_change_code_encoding branch from c6bed8e to a4eacc4 Compare January 16, 2022 14:02
@meltingrabbit
Copy link
Collaborator Author

CI通ったので，マージ．

@meltingrabbit meltingrabbit merged commit 025e083 into develop Jan 16, 2022
@meltingrabbit meltingrabbit deleted the feature/fix_after_change_code_encoding branch January 16, 2022 14:04
@meltingrabbit meltingrabbit self-assigned this Jan 16, 2022
@meltingrabbit
Copy link
Collaborator Author

AddSections忘れとるやんけ．．．

PRだします．

@meltingrabbit meltingrabbit mentioned this pull request Jan 16, 2022
Merged
@meltingrabbit
Copy link
Collaborator Author

#189 で

@meltingrabbit meltingrabbit mentioned this pull request Jan 17, 2022
Merged
@meltingrabbit meltingrabbit mentioned this pull request Feb 7, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yngyu yngyu yngyu approved these changes

@sksat sksat sksat approved these changes

@chutaro chutaro Awaiting requested review from chutaro

Assignees

@meltingrabbit meltingrabbit

Labels
documentation Improvements or additions to documentation enhancement New feature or request priority::high priorityg high S2E tools
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@meltingrabbit @sksat @yngyu