openscad-cpp-evaluator 0.4.6__tar.gz → 0.4.7__tar.gz

{openscad_cpp_evaluator-0.4.6 → openscad_cpp_evaluator-0.4.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openscad_cpp_evaluator
-Version: 0.4.6
+Version: 0.4.7
 Summary: C++ OpenSCAD evaluator with Python bindings
 Author: NeuSCAD Contributors
 License-Expression: MIT

{openscad_cpp_evaluator-0.4.6 → openscad_cpp_evaluator-0.4.7}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "scikit_build_core.build"
 
 [project]
 name = "openscad_cpp_evaluator"
-version = "0.4.6"
+version = "0.4.7"
 description = "C++ OpenSCAD evaluator with Python bindings"
 readme = "README.md"
 requires-python = ">=3.12"

{openscad_cpp_evaluator-0.4.6 → openscad_cpp_evaluator-0.4.7}/src/expr_eval.cpp

@@ -103,7 +103,23 @@ Value Evaluator::evalIdentifier(const std::string& name, const oscad::Position*
     if (const Value* v = ctx.let_->find(name)) return *v;
 
     if (!name.empty() && name[0] == '$') {
-        if (const Value* v = ctx.dyn->find(name)) return *v;
+        // $-prefixed names are dynamic variables: a completely separate
+        // namespace from lexically-scoped identifiers, never requiring a
+        // scope declaration (unlike a bare identifier). Found via
+        // BelfrySCAD/BOSL2's constants.scad `get_slop()`
+        // (`is_undef($slop) ? 0 : $slop` with $slop never assigned
+        // anywhere) producing a spurious "Ignoring unknown variable
+        // '$slop'" warning here but not through the bytecode VM's
+        // Op::LoadDyn (bytecode_vm.cpp), which already does exactly
+        // this -- find in ctx.dyn, undef if absent, no scope-lookup
+        // fallback, no warning, ever. Verified against real OpenSCAD.app:
+        // no warning for a never-set $-variable. Must return here
+        // unconditionally rather than falling through to the
+        // scope->lookupVariable() check below, which can never resolve a
+        // $-name anyway (dynamic variables aren't scope-declarable) and
+        // exists only for plain identifiers.
+        const Value* v = ctx.dyn->find(name);
+        return v ? *v : Value{};
     }
 
     if (name == "PI") return Value{std::numbers::pi};

{openscad_cpp_evaluator-0.4.6 → openscad_cpp_evaluator-0.4.7}/tests/test_expr_eval.cpp

@@ -52,6 +52,23 @@ TEST(ExprEvalIdentifiers, UnknownVariableWarnsAndIsUndef) {
     EXPECT_NE(lastWarning.find("Ignoring unknown variable 'nope'"), std::string::npos);
 }
 
+TEST(ExprEvalIdentifiers, UndefinedDollarVariableIsUndefWithNoWarning) {
+    // Found via BOSL2's constants.scad: `function get_slop() = is_undef($slop)
+    // ? 0 : $slop;` with $slop never assigned anywhere in the script. Real
+    // OpenSCAD.app produces no warning at all -- $-prefixed names are a
+    // separate, always-implicitly-available namespace, never requiring a
+    // scope declaration the way a bare identifier does. This port's
+    // bytecode VM (Op::LoadDyn, bytecode_vm.cpp) already got this right by
+    // construction; the plain interpreter incorrectly fell through to the
+    // same "Ignoring unknown variable" path a genuinely undeclared bare
+    // identifier hits.
+    std::string lastWarning;
+    Evaluator ev([&](const std::string& msg) { lastWarning = msg; });
+    Value v = evalSrc("$slop", ev);
+    EXPECT_TRUE(std::holds_alternative<std::monostate>(v));
+    EXPECT_TRUE(lastWarning.empty());
+}
+
 TEST(ExprEvalIdentifiers, ResolvesViaScopeFallbackWhenNotEagerlyBound) {
     // "a" is a real top-level declaration but evalChildren() was never run,
     // so ctx.let_ has no eager binding for it -- this exercises