Skip to content

Commit 23641a0

Browse files
Avoid checking err.stderr when it's not necessary
1 parent 1b0f0f4 commit 23641a0

File tree

2 files changed

+3
-9
lines changed

2 files changed

+3
-9
lines changed

extensions/ql-vscode/src/codeql-cli/cli.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -547,9 +547,7 @@ export class CodeQLCliServer implements Disposable {
547547
yield JSON.parse(event) as EventType;
548548
} catch (err) {
549549
throw new Error(
550-
`Parsing output of ${description} failed: ${
551-
(err as any).stderr || getErrorMessage(err)
552-
}`,
550+
`Parsing output of ${description} failed: ${getErrorMessage(err)}`,
553551
);
554552
}
555553
}
@@ -647,9 +645,7 @@ export class CodeQLCliServer implements Disposable {
647645
return JSON.parse(result) as OutputType;
648646
} catch (err) {
649647
throw new Error(
650-
`Parsing output of ${description} failed: ${
651-
(err as any).stderr || getErrorMessage(err)
652-
}`,
648+
`Parsing output of ${description} failed: ${getErrorMessage(err)}`,
653649
);
654650
}
655651
}

extensions/ql-vscode/src/common/sarif-parser.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,7 @@ export async function sarifParser(
5454
});
5555
} catch (e) {
5656
throw new Error(
57-
`Parsing output of interpretation failed: ${
58-
(e as any).stderr || getErrorMessage(e)
59-
}`,
57+
`Parsing output of interpretation failed: ${getErrorMessage(e)}`,
6058
);
6159
}
6260
}

0 commit comments

Comments
 (0)