Fix autocomplete issue where open Foo would be picked up inside lin…

…e comments.

Fixes #15.

Changes.md

@@ -1,6 +1,7 @@
 ## master
 - Add support for doc strings when hovering on modules.
 - Add support for printing uncurried function types in hover.
+- Fix autocomplete issue where `open Foo` would be picked up inside line comments (see https://github.com/rescript-lang/rescript-editor-support/issues/15).
 
 ## Release 1.0.3 of rescript-vscode 
 This [commit](https://github.com/rescript-lang/rescript-editor-support/commit/214d220d8573f9f0c8d54e623c163e01617bf124) is vendored in [rescript-vscode 1.0.3](https://github.com/rescript-lang/rescript-vscode/releases/tag/1.0.3).

src/rescript-editor-support/PartialParser.re

@@ -100,6 +100,16 @@ let findCompletable = (text, offset) => {
   };
 };
 
+// Check if the position is inside a `//` comment
+let rec insideLineComment = (text, offset) =>
+  if (offset <= 0 || text.[offset] == '\n') {
+    false;
+  } else if (offset > 0 && text.[offset] == '/' && text.[offset - 1] == '/') {
+    true;
+  } else {
+    insideLineComment(text, offset - 1);
+  };
+
 let findOpens = (text, offset) => {
   let opens = ref([]);
   let pathOfModuleOpen = o => {
@@ -128,7 +138,8 @@ let findOpens = (text, offset) => {
               && text.[at - 3] == 'o'
               && text.[at - 2] == 'p'
               && text.[at - 1] == 'e'
-              && text.[at] == 'n') {
+              && text.[at] == 'n'
+              && !insideLineComment(text, at - 4)) {
             add(String.sub(text, i + 1, i0 + 1 - (i + 1)));
             at - 4;
           } else {