Skip to content

Commit 6dad4da

Browse files
authored
Review/Comment Suggestions are offset by one line if you make local changes first (#6566)
Fixes #6495
1 parent b058a0e commit 6dad4da

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

src/common/diffPositionMapping.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,14 +83,14 @@ export function mapNewPositionToOld(patch: string, line: number): number {
8383
} else {
8484
// Part of the hunk is before line, part is after.
8585
for (const diffLine of diffHunk.diffLines) {
86-
if (diffLine.newLineNumber > line) {
87-
return line + delta;
88-
}
8986
if (diffLine.type === DiffChangeType.Add) {
9087
delta--;
9188
} else if (diffLine.type === DiffChangeType.Delete) {
9289
delta++;
9390
}
91+
if (diffLine.newLineNumber > line) {
92+
return line + delta;
93+
}
9494
}
9595
return line + delta;
9696
}

0 commit comments

Comments
 (0)