libgit2 / libgit2/libgit2sharp
BlameHunk line numbers are zero-based?
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C#
- Estrellas
- 3.5k
- Forks
- 925
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con BlameOptions, BlameHunkCollection.HunkForLine, BlameHunk.ContainsLine y las propiedades FinalStartLineNumber e InitialStartLineNumber. Reproduce la llamada mostrada, compara la indexación documentada con la observada y determina si las APIs deben alinearse o documentarse explícitamente; el trabajo estará terminado cuando el comportamiento sea coherente o esté claramente especificado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp, git
- Área
- api
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100