parallelise calls to nvim gRPC api where possible

idk if this is actually faster or not, gotta test it out Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
kiprasmel · Jun 7, 2022 · 91f6e70 · 91f6e70
1 parent 708cc8e
commit 91f6e70
Showing 1 changed file with 34 additions and 24 deletions.
diff --git a/nvim-git-rebase-todo/nvim-git-rebase-todo.ts b/nvim-git-rebase-todo/nvim-git-rebase-todo.ts
@@ -163,10 +163,11 @@ export default function nvimGitRebaseTodo(plugin: NvimPlugin): void {
 		width, //
 		height,
 	}: SetWindowRelativeToCursorOpts): Promise<OpenWindowOptions> => {
-		const cursor = await vim.window.cursor;
-
-		const relWin = await vim.window;
-		const col: number = config.closeToLeft || (await relWin.width);
+		const [relWin, cursor, col] = await Promise.all([
+			vim.window,
+			vim.window.cursor,
+			config.closeToLeft || vim.window.width,
+		]);
 
 		return {
 			// relative: "cursor",
@@ -195,26 +196,35 @@ export default function nvimGitRebaseTodo(plugin: NvimPlugin): void {
 		 * instead of taking it in as param
 		 */
 
-		const relWin: Window = await vim.getWindow();
-		const col: number = config.closeToLeft || (await relWin.width);
-
-		const opts: OpenWindowOptions = config.relativeToCursor
-			? {
-					...(await getRelativeWindowOptions({ width, height })),
-			  }
-			: ({
-					relative: "win",
-					win: relWin.id,
-					//
-					width,
-					height,
-					//
-					// anchor: "NE", // TODO is this needed?
-					row: 0 + (config.closeToLeft ? config.rowLowerIfCloseToLeft : 0),
-					col,
-					//
-					style: "minimal",
-			  } as const);
+		let relWin: Window;
+		let col: number;
+		let opts: OpenWindowOptions;
+
+		if (config.relativeToCursor) {
+			[relWin, col, opts] = await Promise.all([
+				vim.window, //
+				vim.window.width,
+				getRelativeWindowOptions({ width, height }),
+			]);
+		} else {
+			[relWin, col] = await Promise.all([
+				vim.window, //
+				vim.window.width,
+			]);
+			opts = {
+				relative: "win",
+				win: relWin.id,
+				//
+				width,
+				height,
+				//
+				// anchor: "NE", // TODO is this needed?
+				row: 0 + (config.closeToLeft ? config.rowLowerIfCloseToLeft : 0),
+				col,
+				//
+				style: "minimal",
+			};
+		}
 
 		const enter = false;
 		const window: number | Window = await vim.openWindow(buffer, enter, opts);