[PATCH v8 0/1] MR10362: vbscript: Support chained call syntax like dict.Keys()(i).

[Francis De Brabandere (＠francisdb)]

Treat '(' after tEMPTYBRACKETS as an argument list opener in the
lexer, so that expressions like GetArr()(0) or dict.Keys()(i) parse
correctly. Also dereference VT_BYREF|VT_VARIANT results in vcall so
chained indexing on call results works at runtime.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58051


This includes some of the work done in !10308, preferably that one is merged first.

--
  v8: vbscript: Support chained call syntax like dict.Keys()(i).
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10362

[PATCH v8 1/1] vbscript: Support chained call syntax like dict.Keys()(i).

[Francis De Brabandere]

From: Francis De Brabandere <[e-mail]>

Treat '(' after tEMPTYBRACKETS as an argument list opener in the
lexer, so that expressions like GetArr()(0) or dict.Keys()(i) parse
correctly. Also dereference VT_BYREF|VT_VARIANT results in vcall so
chained indexing on call results works at runtime.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58051
---
 dlls/vbscript/lex.c          |  3 ++-
 dlls/vbscript/tests/lang.vbs | 19 +++++++++++++++++++
 2 files changed, 21 insertions(+), 1 deletion(-)

diff --git a/dlls/vbscript/lex.c b/dlls/vbscript/lex.c
index 12305ab671b..076320e8917 100644
--- a/dlls/vbscript/lex.c
+++ b/dlls/vbscript/lex.c
@@ -491,7 +491,8 @@ static int parse_next_token(void *lval, unsigned *loc, parser_ctx_t *ctx)
          * Parser can't predict if bracket is part of argument expression or an argument
          * in call expression. We predict it here instead.
          */
-        if(ctx->last_token == tIdentifier || ctx->last_token == ')' || ctx->last_token == tME)
+        if(ctx->last_token == tIdentifier || ctx->last_token == ')' || ctx->last_token == tME
+                || ctx->last_token == tEMPTYBRACKETS)
             return '(';
         return tEXPRLBRACKET;
     case '"':
diff --git a/dlls/vbscript/tests/lang.vbs b/dlls/vbscript/tests/lang.vbs
index 39f1d08e27b..1b3d1523c53 100644
--- a/dlls/vbscript/tests/lang.vbs
+++ b/dlls/vbscript/tests/lang.vbs
@@ -1841,6 +1841,25 @@ call ok(containerObj("Key")(1)(0) = "SubValue1", "containerObj(Key)(1)(0) = " &
 
 call ok(Split("1;2", ";")(0) = "1", "Split(""1;2"", "";"")(0) = " & Split("1;2", ";")(0))
 call ok(Split("1;2", ";")(1) = "2", "Split(""1;2"", "";"")(1) = " & Split("1;2", ";")(1))
+Function GetABC()
+    GetABC = Array("a", "b", "c")
+End Function
+call ok(GetABC()(0) = "a", "GetABC()(0) = " & GetABC()(0))
+call ok(GetABC()(1) = "b", "GetABC()(1) = " & GetABC()(1))
+call ok(GetABC()(2) = "c", "GetABC()(2) = " & GetABC()(2))
+
+Function GetNested()
+    GetNested = Array(Array(1, 2), Array(3, 4))
+End Function
+call ok(GetNested()(0)(0) = 1, "GetNested()(0)(0) = " & GetNested()(0)(0))
+call ok(GetNested()(0)(1) = 2, "GetNested()(0)(1) = " & GetNested()(0)(1))
+call ok(GetNested()(1)(0) = 3, "GetNested()(1)(0) = " & GetNested()(1)(0))
+call ok(GetNested()(1)(1) = 4, "GetNested()(1)(1) = " & GetNested()(1)(1))
+
+x = GetABC()(0) & GetABC()(2)
+call ok(x = "ac", "GetABC()(0) & GetABC()(2) = " & x)
+
+call ok(Array(10,20,30)(1) = 20, "Array(10,20,30)(1) = " & Array(10,20,30)(1))
 
 function seta0(arr)
     arr(0) = 2
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/10362

[Vishnunithyasoundhar S (＠svishnunithyasoundhar)]

Vishnunithyasoundhar S (@svishnunithyasoundhar) commented about dlls/vbscript/tests/lang.vbs:
> call ok(Split("1;2", ";")(0) = "1", "Split(""1;2"", "";"")(0) = " & Split("1;2", ";")(0))
>  call ok(Split("1;2", ";")(1) = "2", "Split(""1;2"", "";"")(1) = " & Split("1;2", ";")(1))
> +Function GetABC()
> +    GetABC = Array("a", "b", "c")
> +End Function
> +call ok(GetABC()(0) = "a", "GetABC()(0) = " & GetABC()(0))
> +call ok(GetABC()(1) = "b", "GetABC()(1) = " & GetABC()(1))
> +call ok(GetABC()(2) = "c", "GetABC()(2) = " & GetABC()(2))
> +

Do you think some more tests in the following styles would be nice to add?

`dict.Keys()(0).Length` - dot accessor

`Len(GetStr())` - Chain calls

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10362#note_135511

[Francis De Brabandere (＠francisdb)]

On Wed Apr  8 16:52:00 2026 +0000, Vishnunithyasoundhar S wrote:
> Do you think some more tests in the following styles would be nice to add?
> `dict.Keys()(0).Length` - dot accessor
> `Len(GetStr())` - Chain calls
> Sure, let me have a go at that

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10362#note_135513