Merge pull request #58 from haya14busa/string-subscript

Avoid using string subscript

Author: mattn

autoload/vimlparser.vim

@@ -1226,7 +1226,8 @@ function! s:VimLParser.parse_cmd_function()
 
   if left.type == s:NODE_IDENTIFIER
     let s = left.value
-    if s[0] !=# '<' && !s:isupper(s[0]) && stridx(s, ':') == -1 && stridx(s, '#') == -1
+    let ss = split(s, '\zs')
+    if ss[0] !=# '<' && !s:isupper(ss[0]) && stridx(s, ':') == -1 && stridx(s, '#') == -1
       throw s:Err(printf('E128: Function name must start with a capital or contain a colon: %s', s), left.pos)
     endif
   endif

js/vimlparser.js

@@ -1464,7 +1464,8 @@ VimLParser.prototype.parse_cmd_function = function() {
     this.reader.skip_white();
     if (left.type == NODE_IDENTIFIER) {
         var s = left.value;
-        if (s[0] != "<" && !isupper(s[0]) && viml_stridx(s, ":") == -1 && viml_stridx(s, "#") == -1) {
+        var ss = viml_split(s, "\\zs");
+        if (ss[0] != "<" && !isupper(ss[0]) && viml_stridx(s, ":") == -1 && viml_stridx(s, "#") == -1) {
             throw Err(viml_printf("E128: Function name must start with a capital or contain a colon: %s", s), left.pos);
         }
     }

py/vimlparser.py

@@ -1176,7 +1176,8 @@ def parse_cmd_function(self):
         self.reader.skip_white()
         if left.type == NODE_IDENTIFIER:
             s = left.value
-            if s[0] != "<" and not isupper(s[0]) and viml_stridx(s, ":") == -1 and viml_stridx(s, "#") == -1:
+            ss = viml_split(s, "\\zs")
+            if ss[0] != "<" and not isupper(ss[0]) and viml_stridx(s, ":") == -1 and viml_stridx(s, "#") == -1:
                 raise VimLParserException(Err(viml_printf("E128: Function name must start with a capital or contain a colon: %s", s), left.pos))
         # :function {name}
         if self.reader.peekn(1) != "(":