examples #17
No reviewers
Labels
No labels
bug
dependencies
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
rust
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
navicore/patch-rexx!17
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "phase12c"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
PR Review: Example Files
I reviewed all 15 example REXX files in this PR for correctness, REXX semantics compliance, and potential runtime issues.
Summary
✅ Overall: Examples are well-written and demonstrate key REXX features correctly
All examples are syntactically valid and follow proper REXX semantics. The code demonstrates:
Observations
1. signal_traps.rexx:10 - Division by zero error handling
Observation: According to ANSI X3.274-1996, division by zero should raise Error 42 (Arithmetic overflow/underflow). However, many REXX implementations do trigger SYNTAX conditions for arithmetic errors, so this may be implementation-dependent. The example demonstrates the mechanism correctly, but readers should be aware that the specific error condition triggered by division by zero varies across implementations.
2. parse_demo.rexx:44-45 - Whitespace handling
The concatenation
'Result:' exprproduces'Result: 7'(with a space before 7 due to REXX's blank concatenation operator). The parse correctly extractslabel='Result'andval=' 7'. Line 47 strips the value, so this is handled properly. Just noting that the implicit blank is intentional REXX behavior.3. Comprehensive coverage
The examples provide excellent coverage of:
Recommendation
✅ Approve - These examples are excellent educational resources and demonstrate proper REXX programming patterns. They will be valuable for users learning the interpreter.
No blocking issues found.