Skip to content

Commit 1a87ad8

Browse files
Make comments clearer
1 parent 1317dfa commit 1a87ad8

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

extensions/ql-vscode/src/query-history/query-history-manager.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1086,7 +1086,8 @@ export class QueryHistoryManager extends DisposableObject {
10861086
): Promise<CompletedLocalQueryInfo | undefined> {
10871087
const dbName = fromItem.initialInfo.databaseInfo.name;
10881088

1089-
// if exactly 2 queries are selected, use those
1089+
// If exactly 2 items are selected, return the one that
1090+
// isn't being used as the "from" item.
10901091
if (allItemsSelected.length === 2) {
10911092
const otherItem =
10921093
fromItem === allItemsSelected[0]
@@ -1102,7 +1103,7 @@ export class QueryHistoryManager extends DisposableObject {
11021103
throw new Error("Please select no more than 2 queries.");
11031104
}
11041105

1105-
// otherwise, let the user choose
1106+
// Otherwise, present a dialog so the user can choose the item they want to use.
11061107
const comparableQueryLabels = this.treeDataProvider.allHistory
11071108
.filter(this.isSuccessfulCompletedLocalQueryInfo)
11081109
.filter(

0 commit comments

Comments
 (0)