From f500653cece7dbd250b7fbd664dcf628755ecdfb Mon Sep 17 00:00:00 2001 From: Chris Lemaire Date: Tue, 28 Oct 2025 11:53:38 +0100 Subject: [PATCH] [gh] fix off-by-one in changed ranges mapping (IJPL-214498) #IJPL-214498 Fixed (cherry picked from commit 74698994d4fdc22bccc198c5134802a8c2f9aea6) IJ-CR-182230 GitOrigin-RevId: e0634ec3351b17cd72254e84c19a9f5b4ca5fcef --- .../github/pullrequest/ui/diff/GHPRReviewDiffExtension.kt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/plugins/github/github-core/src/org/jetbrains/plugins/github/pullrequest/ui/diff/GHPRReviewDiffExtension.kt b/plugins/github/github-core/src/org/jetbrains/plugins/github/pullrequest/ui/diff/GHPRReviewDiffExtension.kt index bbae08552904..15a28dc4142a 100644 --- a/plugins/github/github-core/src/org/jetbrains/plugins/github/pullrequest/ui/diff/GHPRReviewDiffExtension.kt +++ b/plugins/github/github-core/src/org/jetbrains/plugins/github/pullrequest/ui/diff/GHPRReviewDiffExtension.kt @@ -192,7 +192,9 @@ private class DiffEditorModel( Side.LEFT -> range.end1 Side.RIGHT -> range.end2 } - val end = locationToLine(side to endLineIdx.dec())?.inc() ?: return null + val end = + if (startLineIdx == endLineIdx) start + else locationToLine(side to endLineIdx.dec())?.inc() ?: return null return LineRange(start, end) }