Skip to content

Commit b434d42

Browse files
committed
Rename ParenthesizedExprSynth to StmtSequenceSynth
1 parent ff06e72 commit b434d42

3 files changed

Lines changed: 18 additions & 22 deletions

File tree

ql/src/codeql_ruby/ast/Expr.qll

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,12 @@ class StmtSequence extends Expr, TStmtSequence {
7979
override AstNode getAChild(string pred) { pred = "getStmt" and result = this.getStmt(_) }
8080
}
8181

82+
private class StmtSequenceSynth extends StmtSequence, TStmtSequenceSynth {
83+
final override Stmt getStmt(int n) { synthChild(this, n, result) }
84+
85+
final override string toString() { result = "..." }
86+
}
87+
8288
private class Then extends StmtSequence, TThen {
8389
private Generated::Then g;
8490

@@ -206,21 +212,15 @@ class BodyStmt extends StmtSequence, TBodyStmt {
206212
* ```
207213
*/
208214
class ParenthesizedExpr extends StmtSequence, TParenthesizedExpr {
209-
final override string getAPrimaryQlClass() { result = "ParenthesizedExpr" }
210-
211-
final override string toString() { result = "( ... )" }
212-
}
213-
214-
private class ParenthesizedExprReal extends ParenthesizedExpr, TParenthesizedExprReal {
215215
private Generated::ParenthesizedStatements g;
216216

217-
ParenthesizedExprReal() { this = TParenthesizedExprReal(g) }
217+
ParenthesizedExpr() { this = TParenthesizedExpr(g) }
218218

219219
final override Stmt getStmt(int n) { toGenerated(result) = g.getChild(n) }
220-
}
221220

222-
private class ParenthesizedExprSynth extends ParenthesizedExpr, TParenthesizedExprSynth {
223-
final override Stmt getStmt(int n) { synthChild(this, n, result) }
221+
final override string getAPrimaryQlClass() { result = "ParenthesizedExpr" }
222+
223+
final override string toString() { result = "( ... )" }
224224
}
225225

226226
/**

ql/src/codeql_ruby/ast/internal/AST.qll

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -188,10 +188,7 @@ private module Cached {
188188
TNotExpr(Generated::Unary g) { g instanceof @unary_bang or g instanceof @unary_not } or
189189
TOptionalParameter(Generated::OptionalParameter g) or
190190
TPair(Generated::Pair g) or
191-
TParenthesizedExprReal(Generated::ParenthesizedStatements g) or
192-
TParenthesizedExprSynth(AST::AstNode parent, int i) {
193-
mkSynthChild(ParenthesizedExprKind(), parent, i)
194-
} or
191+
TParenthesizedExpr(Generated::ParenthesizedStatements g) or
195192
TRShiftExprReal(Generated::Binary g) { g instanceof @binary_ranglerangle } or
196193
TRShiftExprSynth(AST::AstNode parent, int i) { mkSynthChild(RShiftExprKind(), parent, i) } or
197194
TRangeLiteral(Generated::Range g) or
@@ -231,6 +228,7 @@ private module Cached {
231228
TSpaceshipExpr(Generated::Binary g) { g instanceof @binary_langleequalrangle } or
232229
TSplatArgument(Generated::SplatArgument g) or
233230
TSplatParameter(Generated::SplatParameter g) or
231+
TStmtSequenceSynth(AST::AstNode parent, int i) { mkSynthChild(StmtSequenceKind(), parent, i) } or
234232
TStringArrayLiteral(Generated::StringArray g) or
235233
TStringConcatenation(Generated::ChainedString g) or
236234
TStringEscapeSequenceComponent(Generated::EscapeSequence g) or
@@ -369,7 +367,7 @@ private module Cached {
369367
n = TNotExpr(result) or
370368
n = TOptionalParameter(result) or
371369
n = TPair(result) or
372-
n = TParenthesizedExprReal(result) or
370+
n = TParenthesizedExpr(result) or
373371
n = TRShiftExprReal(result) or
374372
n = TRangeLiteral(result) or
375373
n = TRationalLiteral(result) or
@@ -484,15 +482,15 @@ private module Cached {
484482
kind = MulExprKind() and
485483
result = TMulExprSynth(parent, i)
486484
or
487-
kind = ParenthesizedExprKind() and
488-
result = TParenthesizedExprSynth(parent, i)
489-
or
490485
kind = RShiftExprKind() and
491486
result = TRShiftExprSynth(parent, i)
492487
or
493488
kind = SelfKind() and
494489
result = TSelfSynth(parent, i)
495490
or
491+
kind = StmtSequenceKind() and
492+
result = TStmtSequenceSynth(parent, i)
493+
or
496494
kind = SubExprKind() and
497495
result = TSubExprSynth(parent, i)
498496
)
@@ -560,9 +558,7 @@ class TExpr =
560558

561559
class TStmtSequence =
562560
TBeginBlock or TEndBlock or TThen or TElse or TDo or TEnsure or TStringInterpolationComponent or
563-
TBlock or TBodyStmt or TParenthesizedExpr;
564-
565-
class TParenthesizedExpr = TParenthesizedExprReal or TParenthesizedExprSynth;
561+
TBlock or TBodyStmt or TParenthesizedExpr or TStmtSequenceSynth;
566562

567563
class TBodyStmt = TBeginExpr or TModuleBase or TMethod or TLambda or TDoBlock or TSingletonMethod;
568564

ql/src/codeql_ruby/ast/internal/Synthesis.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ newtype SynthKind =
2525
LogicalOrExprKind() or
2626
ModuloExprKind() or
2727
MulExprKind() or
28-
ParenthesizedExprKind() or
28+
StmtSequenceKind() or
2929
RShiftExprKind() or
3030
SelfKind() or
3131
SubExprKind()

0 commit comments

Comments
 (0)