fix selection order of prefix/postfix AST fields

Added tests to the existing suite.
NomicFoundation · Feb 23, 2024 · f5f1e40 · f5f1e40
1 parent ad98d1c
commit f5f1e40
Show file tree

Hide file tree

Showing 12 changed files with 583 additions and 93 deletions.
diff --git a/.changeset/lemon-knives-fry.md b/.changeset/lemon-knives-fry.md
@@ -0,0 +1,5 @@
+---
+"@nomicfoundation/slang": patch
+---
+
+fix selection order of prefix/postfix AST fields
diff --git a/crates/codegen/parser/generator/src/ast_model.rs b/crates/codegen/parser/generator/src/ast_model.rs
@@ -221,10 +221,10 @@ impl AstModel {
 
         match operator.model {
             model::OperatorModel::Prefix => {
-                fields.insert(0, operand("operand"));
+                fields.push(operand("operand"));
             }
             model::OperatorModel::Postfix => {
-                fields.push(operand("operand"));
+                fields.insert(0, operand("operand"));
             }
             model::OperatorModel::BinaryLeftAssociative
             | model::OperatorModel::BinaryRightAssociative => {

diff --git a/crates/solidity/outputs/cargo/slang_solidity/src/generated/napi_interface/ast_selectors.rs b/crates/solidity/outputs/cargo/slang_solidity/src/generated/napi_interface/ast_selectors.rs