libgit2 / libgit2/libgit2sharp
BlameHunk line numbers are zero-based?
Nessuno ha ancora preso questa issue.
- Lingua principale
- C#
- Stelle
- 3.5k
- Fork
- 925
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia con BlameOptions, BlameHunkCollection.HunkForLine, BlameHunk.ContainsLine e le proprietà FinalStartLineNumber e InitialStartLineNumber. Riproduci la chiamata mostrata, confronta l’indicizzazione documentata con quella osservata e determina se le API debbano essere allineate o documentate esplicitamente; il lavoro è completo quando il comportamento è coerente o chiaramente specificato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- csharp, git
- Ambito
- api
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 38/100