From c019da83f3acdd97ab1604b1d457ffeec29ebc30 Mon Sep 17 00:00:00 2001 From: Nick Rolfe Date: Wed, 17 Feb 2021 10:57:01 +0000 Subject: [PATCH] Address feedback on StringInterpolationComponent::getStmt --- ql/src/codeql_ruby/ast/internal/Literal.qll | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/ql/src/codeql_ruby/ast/internal/Literal.qll b/ql/src/codeql_ruby/ast/internal/Literal.qll index e0e1284f4cd..b8ef438b70b 100644 --- a/ql/src/codeql_ruby/ast/internal/Literal.qll +++ b/ql/src/codeql_ruby/ast/internal/Literal.qll @@ -119,9 +119,8 @@ module StringInterpolationComponent { final override string toString() { result = "#{...}" } - final override Expr getStmt(int n) { - // TODO: fix grammar to properly handle a sequence of more than one expr, - // e.g. #{ foo; bar } + final override Stmt getStmt(int n) { + // Generated AST can currently only represent a single statement in an interpolation. n = 0 and result = generated.getChild() }