From 48612871384cfcf1756f871da244d06ce331911b Mon Sep 17 00:00:00 2001 From: cnc Date: Tue, 2 Jun 2026 15:54:17 +0800 Subject: [PATCH] =?UTF-8?q?=E7=BB=A7=E7=BB=AD=E6=8C=89=20align-linuxcnc=20?= =?UTF-8?q?=E7=BA=A6=E6=9D=9F=EF=BC=9A=E8=A1=A5=E8=A1=A8=E8=BE=BE=E5=BC=8F?= =?UTF-8?q?=E9=94=99=E8=AF=AF=E6=BA=90=E7=A0=81=E5=9B=9E=E5=BD=92?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/linuxcnc-porting.md | 2 +- test-linuxcnc-source-link.sh | 19 +++++++++++++++++++ tests/gcode/linuxcnc_divide_by_zero_error.ngc | 2 ++ tests/gcode/linuxcnc_negative_power_error.ngc | 2 ++ 4 files changed, 24 insertions(+), 1 deletion(-) create mode 100644 tests/gcode/linuxcnc_divide_by_zero_error.ngc create mode 100644 tests/gcode/linuxcnc_negative_power_error.ngc diff --git a/docs/linuxcnc-porting.md b/docs/linuxcnc-porting.md index 9eb69ac..4529016 100644 --- a/docs/linuxcnc-porting.md +++ b/docs/linuxcnc-porting.md @@ -212,6 +212,6 @@ This matrix tracks LinuxCNC feature coverage for the web/WASM simulator. A featu | Cutter compensation | covered for tool-table `G41/G42 D...` and explicit-radius `G41.1/G42.1/G40` through LinuxCNC native/source backends | `tests/gcode/linuxcnc_cutter_comp.ngc` | | Probe moves `G38.2/G38.3/G38.4/G38.5` | covered as distinct probe events with LinuxCNC `probe_type` preserved through smoke, native, and source-linked paths | `tests/gcode/linuxcnc_probe_no_error.ngc` | | O-word subroutines, calls, and control flow | covered for numeric and named `O... sub/call/return/endsub` plus `if`/`elseif`/`else`, `while`, `repeat`, `do`/`while`, `break`, and `continue` through LinuxCNC file mode; smoke parser also covers named `O` sub/call/return/endsub and loop-control forms | `tests/gcode/smoke_oword_subprogram.ngc`, `tests/gcode/linuxcnc_named_oword_subprogram.ngc`, `tests/gcode/linuxcnc_oword_control_flow.ngc` | -| Parameter expressions and operators | covered for parameter references and source-linked `mod`, `**`, logical, and relational operator precedence from LinuxCNC `interp_read.cc`/`interp_execute.cc` | `tests/gcode/linuxcnc_parameter_expression_assignment.ngc`, `tests/gcode/linuxcnc_expression_operators.ngc` | +| Parameter expressions and operators | covered for parameter references, source-linked `mod`, `**`, logical, and relational operator precedence, plus LinuxCNC divide-by-zero and negative-power error paths from `interp_read.cc`/`interp_execute.cc` | `tests/gcode/linuxcnc_parameter_expression_assignment.ngc`, `tests/gcode/linuxcnc_expression_operators.ngc`, `tests/gcode/linuxcnc_divide_by_zero_error.ngc`, `tests/gcode/linuxcnc_negative_power_error.ngc` | | Broader canned cycles `G73`, `G74`, `G82`-`G89` | partially covered: `G73`, `G74`, `G82`, `G83`, `G84`, `G85`, `G86`, `G87`, `G88`, `G89`; smoke parser maps supported cycles across `G17/G18/G19`, including LinuxCNC's special non-XY `G74/G84` tapping argument order and `G87` plane-specific `I/J/K` mapping; native/source regressions now check non-XY `G74/G84/G86/G87/G88` sequences | `tests/gcode/linuxcnc_canned_cycles_extended.ngc`, `tests/gcode/linuxcnc_canned_cycle_planes.ngc`, smoke API regression | | Full source-level wasm build | covered for the current wasm-safe RS274 source set, with remaining work focused on broadening browser-hostile dependency replacement and feature coverage | `./build-wasm.sh`, `test-linuxcnc-wasm-runtime-link.sh`, `test-web-wasm-node-smoke.sh`, `test-web-wasm-browser-smoke.sh` | diff --git a/test-linuxcnc-source-link.sh b/test-linuxcnc-source-link.sh index f6bb31e..aff10e0 100755 --- a/test-linuxcnc-source-link.sh +++ b/test-linuxcnc-source-link.sh @@ -571,6 +571,25 @@ if CNC_SIM_RS274_FILE_MODE=1 CNC_SIM_RS274_VAR="$var_file" \ echo "expected linuxcnc_infinite_expression_error.ngc to fail" >&2 exit 1 fi +# Source basis: LinuxCNC src/emc/rs274ngc/interp_execute.cc +# execute_binary1() rejects divide-by-zero and negative base +# non-integer power expression operators. +if CNC_SIM_RS274_FILE_MODE=1 CNC_SIM_RS274_VAR="$var_file" \ + "$build_dir/linuxcnc_rs274_source_dump" tests/gcode/linuxcnc_divide_by_zero_error.ngc \ + >"$output_dir/cnc_sim_linuxcnc_source_divide_by_zero_error.json" 2>"$output_dir/cnc_sim_linuxcnc_source_divide_by_zero_error.err"; then + echo "expected linuxcnc_divide_by_zero_error.ngc to fail" >&2 + exit 1 +fi +grep -q "Attempt to divide by zero" \ + "$output_dir/cnc_sim_linuxcnc_source_divide_by_zero_error.err" +if CNC_SIM_RS274_FILE_MODE=1 CNC_SIM_RS274_VAR="$var_file" \ + "$build_dir/linuxcnc_rs274_source_dump" tests/gcode/linuxcnc_negative_power_error.ngc \ + >"$output_dir/cnc_sim_linuxcnc_source_negative_power_error.json" 2>"$output_dir/cnc_sim_linuxcnc_source_negative_power_error.err"; then + echo "expected linuxcnc_negative_power_error.ngc to fail" >&2 + exit 1 +fi +grep -q "Attempt to raise negative to non integer power" \ + "$output_dir/cnc_sim_linuxcnc_source_negative_power_error.err" CNC_SIM_RS274_FILE_MODE=1 CNC_SIM_RS274_VAR="$var_file" \ "$build_dir/linuxcnc_rs274_source_dump" tests/gcode/linuxcnc_value_returned.ngc \ >"$output_dir/cnc_sim_linuxcnc_source_value_returned.json" diff --git a/tests/gcode/linuxcnc_divide_by_zero_error.ngc b/tests/gcode/linuxcnc_divide_by_zero_error.ngc new file mode 100644 index 0000000..4cca6c2 --- /dev/null +++ b/tests/gcode/linuxcnc_divide_by_zero_error.ngc @@ -0,0 +1,2 @@ +#1 = [1 / 0] +G1 X#1 F100 diff --git a/tests/gcode/linuxcnc_negative_power_error.ngc b/tests/gcode/linuxcnc_negative_power_error.ngc new file mode 100644 index 0000000..2afd72d --- /dev/null +++ b/tests/gcode/linuxcnc_negative_power_error.ngc @@ -0,0 +1,2 @@ +#1 = [-2 ** 0.5] +G1 X#1 F100