Fixed get_code_region_around_line() boundary issue #81
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In the
get_code_region_around_line()
function,line_no
is 1-based, so we need to ensure that the last valid line is included in the output.If
line_no + window_size
exceedslen(file_content)
, the current implementation excludes the last line due to the exclusive nature ofrange(start, end)
.To fix this issue, I added
1
tolen(file_content)
, ensuring that the last line is included.Additionally,
i
inrange(start, end)
is 1-based becausestart
andend
are derived fromline_no
, which follows 1-based indexing.Without adjusting the index in the
else
block, usingfile_content[i]
would be off by one, potentially leading to anIndexError
or retrieving the wrong lines.To fix this, I replaced
file_content[i]
withfile_content[i - 1]