Only remove unwanted character at the beginning of a string (#51). (#55)

mt40 · Mar 18, 2017 · 88a5a2b · 88a5a2b
1 parent 1f2733f
commit 88a5a2b
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,6 @@
+## 1.8.8
+- Fix a bug where asterisk(*) and slash(/) are removed even if they are in the middle of a string.
+
 ## 1.8.7
 - Add support for Matlab.
 

diff --git a/package.json b/package.json
@@ -2,7 +2,7 @@
   "name": "vscode-todo-parser",
   "displayName": "TODO Parser",
   "description": "Parse TODOs in your working files.",
-  "version": "1.8.7",
+  "version": "1.8.8",
   "publisher": "minhthai",
   "engines": {
     "vscode": "^1.2.0"

diff --git a/src/classes/Parser.ts b/src/classes/Parser.ts
@@ -114,9 +114,9 @@ export class Parser {
    */
   private static cleanString(str: string): string {
     let no_space = str.trim();
-    let no_leading_slash = no_space.replace(/\/+/, '');
-    let no_leading_asterisk = no_leading_slash.replace(/\*+/g, '');
-    no_leading_asterisk = no_leading_asterisk.replace(/\/+/, ''); // remove slash again!
+    let no_leading_slash = no_space.replace(/^\/+/, '');
+    let no_leading_asterisk = no_leading_slash.replace(/^\*+/g, '');
+    no_leading_asterisk = no_leading_asterisk.replace(/^\/+/, ''); // remove slash again!
     str = no_leading_asterisk.trim();
     return str;
   }

diff --git a/test/samples/spaced path/cpp.cpp b/test/samples/spaced path/cpp.cpp
@@ -4,6 +4,7 @@ using namespace std;
 
 int main() {
   //TODO: cpp one line
+  // TODO scale axis to get metres/feet
   /* Todo: cpp multiline
   * todo 
   */