fix: show cmd-line errors in status bar. add new E492 error

src/cmd_line/commandLine.ts

@@ -6,6 +6,7 @@ import { VimState } from '../state/vimState';
 import { StatusBar } from '../statusBar';
 import * as parser from './parser';
 import * as util from '../util';
+import { VimError, ErrorCode } from '../error';
 
 export class CommandLine {
   public static async PromptAndRun(initialText: string, vimState: VimState): Promise<void> {
@@ -37,8 +38,12 @@ export class CommandLine {
         await cmd.execute(vimState.editor, vimState);
       }
     } catch (e) {
-      console.error(e);
-      await util.showError(`${e.message}`);
+      console.log(e);
+      if (e instanceof VimError) {
+        StatusBar.SetText(`${e.toString()}: ${command}`, vimState.currentMode, true);
+      } else {
+        util.showError(e.toString());
+      }
     }
   }
 

src/cmd_line/parser.ts

@@ -2,6 +2,7 @@ import * as lexer from './lexer';
 import * as node from './node';
 import { commandParsers } from './subparser';
 import * as token from './token';
+import { VimError, ErrorCode } from '../error';
 
 interface IParseFunction {
   (state: ParserState, command: node.CommandLine): IParseFunction | null;
@@ -52,7 +53,7 @@ function parseCommand(state: ParserState, commandLine: node.CommandLine): IParse
       case token.TokenType.CommandName:
         var commandParser = (commandParsers as any)[tok.content];
         if (!commandParser) {
-          throw new Error('Not implemented or not a valid command');
+          throw VimError.fromCode(ErrorCode.E492);
         }
         // TODO: Pass the args, but keep in mind there could be multiple
         // commands, not just one.

src/error.ts

@@ -10,6 +10,7 @@ export enum ErrorCode {
   E208 = 208,
   E348 = 348,
   E444 = 444,
+  E492 = 492,
   E488 = 488,
 }
 
@@ -19,6 +20,7 @@ export const ErrorMessage: IErrorMessage = {
   208: 'Error writing to file',
   348: 'No string under cursor',
   444: 'Cannot close last window',
+  492: 'Not an editor command',
   488: 'Trailing characters',
 };
 
@@ -48,10 +50,6 @@ export class VimError extends Error {
     return this._message;
   }
 
-  display(): void {
-    util.showError(this.toString());
-  }
-
   toString(): string {
     return 'E' + this.code.toString() + ': ' + this.message;
   }

src/mode/modeHandler.ts

@@ -266,15 +266,15 @@ export class ModeHandler implements vscode.Disposable {
   async handleKeyEvent(key: string): Promise<Boolean> {
     const now = Number(new Date());
 
-    // Rewrite commands.
-    // The conditions when you trigger a "copy" rather than a ctrl-c are
-    // too sophisticated to be covered by the "when" condition in package.json
+    // Rewrite commands
     if (Configuration.overrideCopy) {
+      // The conditions when you trigger a "copy" rather than a ctrl-c are
+      // too sophisticated to be covered by the "when" condition in package.json
       if (key === '<D-c>') {
         key = '<copy>';
       }
 
-      if (process.platform !== 'darwin' && key === '<C-c>') {
+      if (key === '<C-c>' && process.platform !== 'darwin') {
         if (
           !Configuration.useCtrlKeys ||
           this.vimState.currentMode === ModeName.Visual ||
@@ -1344,6 +1344,7 @@ export class ModeHandler implements vscode.Disposable {
       // Update all EasyMotion decorations
       this.vimState.easyMotion.updateDecorations();
     }
+
     this._renderStatusBar();
 
     await vscode.commands.executeCommand(
@@ -1385,7 +1386,8 @@ export class ModeHandler implements vscode.Disposable {
       text.push(macroText);
     }
 
-    StatusBar.SetText(text.join(' '), this.currentMode.name, true);
+    let forceUpdate = this.currentMode.name === ModeName.SearchInProgressMode;
+    StatusBar.SetText(text.join(' '), this.currentMode.name, forceUpdate);
   }
 
   async handleMultipleKeyEvents(keys: string[]): Promise<void> {