Unit Test expecting 57, 123; code review
まだ誰も着手していません。
評価
調査の方向性
antlr4cpp/antlr/test/test_interval_set.cpp と test_mixed_ranges_and_elements() メソッドから始めます。期待される interval string を、示されている Java 実装と比較し、関連する C++ unit test を実行します。テストの期待値または interval の動作が確認され、意図された inclusive-range semantics と一致していれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Hello,
Inspiring on C ++ test for my unit tests in C, I noticed an error in the expecting result. Maybe the +1 has no effect?
In https://github.com/antlr/antlr4-cpp/blob/master/antlr4cpp/antlr/test/test_interval_set.cpp
void test_mixed_ranges_and_elements()
{
interval_set s;
s.insert(1);
s.insert(std::make_pair(L'a',L'z'+1));
s.insert(std::make_pair(L'0',L'9'+1));
std::wstring expecting = L"{1, 48..57, 97..122}";
}
by comparison with Java implementation
IntervalSet interval_set = new IntervalSet();
interval_set.add(1);
interval_set.add('a', 'z'+1);
interval_set.add('0', '9'+1);
String str = interval_set.toString();
String expecting = "{1, 48..58, 97..123}";
PS:
C ++ operator overloading is friendly
interval_set s;
s << 1;// s+1 is ambigus
s << interval('a', 'z'+1);
s &= interval('0', '9'+1);
s |= interval('0', '9'+1);
std::cout << s << std::endl;
- 主要言語
- C++
- スター
- 80
- フォーク
- 33
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
antlr/antlr4-cpp のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 30/100
antlr/antlr4-cpp#14 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
antlr/antlr4-cpp#13 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
antlr/antlr4-cpp#12 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
antlr/antlr4-cpp#11 ·
antlr/antlr4-cpp の issue をすべて見る
似ている issue
-
bug tooling
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 69/100
メンテナーはふだん 1 日以内に返信
-
bug GPU
難易度 2/5 1〜3時間 初心者へのやさしさ 69/100
AMReX-Astro/Microphysics#2115 ·
メンテナーはふだん 1 日以内に返信
-
Bug: negative DECIMAL with fewer digits than its scale prints as "0.-5"対応中かも @adsharma が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
LadybugDB/ladybug#1311 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信