UTF32BEの文字コード変換クラスが7bit ASCIIを取り込めない問題に対処する #1627
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
テストを追加しただけだとエラーになりました。 |
|
Kudos, SonarCloud Quality Gate passed! |
|
✅ Build sakura 1.0.3672 completed (commit 3654279718 by @berryzplus) |
kengoide
approved these changes
Apr 15, 2021
| @@ -451,6 +451,54 @@ TEST(CCodeBase, codeUtf32Le) | |||
|
|
|||
| //! googletestの出力に文字セットIDを出力させる | |||
| std::ostream& operator << (std::ostream& os, const ECodeType& eCodeType); | |||
| /*! | |||
There was a problem hiding this comment.
・・・どちらかというと、テストを挿入する場所を間違ってますね。
普通に考えて、452行目の手前に入れるのが適切な気がします。
このPRはこのまま進めて、なんかのついでに直してしまおうと思います。
|
レビューありがとうございます。マージしちゃいます。 挿入箇所を間違えてる件については、次回このテストファイルを触るときに対応したいと思います。 |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
タイトル通りです。少なくとも 7bit ASCII を取り込めるようにします。
カテゴリ
PR の背景
#1614 (comment) で書いた通り、UTF32BEの文字コード変換クラスに不具合が見つかっています。
PR のメリット
PR のデメリット (トレードオフとかあれば)
修正コード周辺には不審点がいくつかあるので、直しても結局動かないかも知れません。
仕様・動作説明
PR の影響範囲
テスト内容
コード修正を行う前は、このテストに失敗します。
コード修正を行った後は、このテストに成功します。
関連 issue, PR
参考資料