Potential false negatives on protected methods referenced from child class
まだ誰も着手していません。
評価
調査の方向性
リンクされた pull request の議論と、この issue で説明されている Linkage Checker の動作を確認してから、子クラスと親クラスの関係における protected メソッドのケースを再現します。完了条件は、ライブラリメソッドが public から protected に変更されたときに Linkage Checker が linkage error を報告し、class-file シナリオのカバレッジがあることです。
索引モデルが issue の本文から書いたものです。
説明
https://github.com/GoogleCloudPlatform/cloud-opensource-java/pull/2183#discussion_r688009905 found an interesting case where Linkage Checker cannot report linkage errors for protected methods that are referenced from a child class. For example, the code below shows red for compilation error but Linkage Checker would not detect it if the relationship is in a class files.
This implies that when
- a method in a library is changing its accessor from public to protected, and
- the caller and callee has child-parent class relationship
Linkage Checker would fail to report the linkage errors. It considers that a protected method is ok to be accessed by child classes.
I'm not sure how to fix it but I think it's worth document here.
- 主要言語
- Java
- スター
- 163
- フォーク
- 81
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
GoogleCloudPlatform/cloud-opensource-java のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
GoogleCloudPlatform/cloud-opensource-java#2419 · コメント 1 件 · リアクション 2 件 ·
-
best practices
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 42/100
-
enhancement p2
GoogleCloudPlatform/cloud-opensource-java#2468 · コメント 1 件 · 担当者 1 名 ·
GoogleCloudPlatform/cloud-opensource-java の issue をすべて見る
似ている issue
-
awaiting triage bug Causes friction Hop Gui P1 P2 Transforms
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
apache/flink-agents#1152 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
jenkinsci/blueocean-plugin#5417 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
objectionary/eo-graphs#75 ·