bugfix:manage operations when using MSDOSplayer

Author: xsro

package.json

@@ -3,7 +3,7 @@
   "displayName": "MASM/TASM",
   "publisher": "xsro",
   "description": "DOS汇编语言开发环境和语法支持，win下安装即用,自带DOSBox.exe和TASM/MASM工具",
-  "version": "0.5.2",
+  "version": "0.5.3-beta.1",
   "keywords": [
     "dosbox",
     "16位",

src/DOSBox.ts

@@ -1,4 +1,4 @@
-import { Uri, workspace, window, TextDocument, FileType } from 'vscode';
+import { Uri, workspace, window, TextDocument } from 'vscode';
 import { Config } from './configration';
 import { exec, ExecOptions } from 'child_process';
 /**
@@ -126,7 +126,6 @@ function openDosbox(opt: OPTS): Promise<string> {
                     }
                     window.showErrorMessage(msg);
                 }
-
             })
         }
     );

src/configration.ts

@@ -1,4 +1,4 @@
-import { workspace, window, Uri, FileSystem, FileType, ExtensionContext } from 'vscode';
+import { workspace, Uri, FileSystem, FileType, ExtensionContext } from 'vscode';
 import { TextEncoder } from 'util';
 interface ToolInfo {
     uri: Uri,

src/runcode.ts

@@ -60,7 +60,9 @@ export class AsmAction {
             }
             if (goon && (DOSemu === "msdos player" || DOSemu === "auto")) {
                 let flag: boolean = DOSemu === "msdos player";
-                flag = (MASMorTASM === "masm" || runOrDebug === false || DOSemu == "auto") || flag
+                //msdos mode:  TASM debug command `TD` can only run in dosbox;(I do this inside `MSDos.RunDebug`)
+                //auto mode: run in dosbox,`TD` in dosbox,MASM debug command `debuq` in player
+                flag = (MASMorTASM === "MASM" && runOrDebug === false && DOSemu == "auto") || flag
                 MSDos.RunDebug(this._config, flag, runOrDebug);
             };
         }

src/viaPlayer.ts

@@ -1,4 +1,4 @@
-import { window, Terminal, Uri, TextDocument } from 'vscode';
+import { window, Terminal } from 'vscode';
 import { Config } from './configration';
 import { exec } from 'child_process';
 import * as DOSBox from './DOSBox';
@@ -17,6 +17,20 @@ export function runPlayer(conf: Config, filename: string): Promise<string> {
                     }
                 }
             );
+            child.on('exit', (code) => {
+                if (code && code !== 0) {
+                    let msg = `Use playerasm.bat Failed\t exitcode${code}\t\nFilepath: ${conf.msbatpath}`;
+                    window.showErrorMessage(msg);
+                }
+            });
+            let timeout: number = 3000;
+            setTimeout(() => {
+                if (child.exitCode === null) {
+                    child.kill();
+                    window.showErrorMessage(`Run playerasm.bat timeout after ${timeout}ms\t\nFilepath: ${conf.msbatpath} `)
+                }
+            }, timeout);
+
         }
     );