Reduce VS Code requirement. (#3241)

* Reduce VS Code requirement. * Remove more addresses in crash telemetry.
microsoft · Mar 2, 2019 · 95dd538 · 95dd538
1 parent 5512035
commit 95dd538
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Extension/package.json b/Extension/package.json
@@ -12,7 +12,7 @@
   },
   "license": "SEE LICENSE IN LICENSE.txt",
   "engines": {
-    "vscode": "^1.31.0"
+    "vscode": "^1.30.0"
   },
   "bugs": {
     "url": "https://github.com/Microsoft/vscode-cpptools/issues",

diff --git a/Extension/src/LanguageServer/extension.ts b/Extension/src/LanguageServer/extension.ts
@@ -859,6 +859,7 @@ function handleCrashFileRead(err: NodeJS.ErrnoException, data: string): void {
 
     // Get rid of the memory addresses (which breaks being able get a hit count for each crash call stack).
     data = data.replace(/0x................ /g, "");
+    data = data.replace(/0x1........ \+ 0/g, "");
 
     // Get rid of the process names on each line and just add it to the start.
     const process1: string = "Microsoft.VSCode.CPP.IntelliSense.Msvc.darwin\t";