libgit2 / libgit2/libgit2sharp
BlameHunk line numbers are zero-based?
まだ誰も着手していません。
- 主要言語
- C#
- スター
- 3.5k
- フォーク
- 925
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
var x = 5; // just an example
var options = new BlameOptions {MinLine = x, MaxLine = x};
var hunks = repository.Blame(path, options);
var hunk = hunks.HunkForLine(x); // throws ArgumentOutOfRangeException: 'No hunk for that line'
It seems that though the line numbers in BlameOptions are one-based (according to the code comments), BlameHunkCollection.HunkForLine, BlameHunk.ContainsLine, BlameHunk.FinalStartLineNumber and BlameHunk.InitialStartLineNumber are all zero based.
Is this a bug, or intentional? If intentional, it would be good to add to the comments on all of these which base is to be expected. Thanks.
I can subtract one in the HunkForLine call to work around for now.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
BlameOptions、BlameHunkCollection.HunkForLine、BlameHunk.ContainsLine、および FinalStartLineNumber と InitialStartLineNumber のプロパティから始めます。示されている呼び出しを再現し、ドキュメントに記載されたインデックスと観測されたインデックスを比較して、API を揃えるべきか、明示的に文書化すべきかを判断します。動作に一貫性があるか、明確に仕様化されていれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- csharp, git
- 領域
- api
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 38/100