Skip to content

PCBGeneric: read Arc.LineWidth, not Arc.Width, in the Width property getter - #30

Closed
bblacklock3 wants to merge 1 commit into
salitronic:mainfrom
bblacklock3:fix/arc-linewidth
Closed

bblacklock3 wants to merge 1 commit into
salitronic:mainfrom
bblacklock3:fix/arc-linewidth

Conversation

@bblacklock3

Copy link
Copy Markdown
Contributor

IPCB_Arc has no Width member, so obj_query with Width in the property list on an eArcObject raised a modal Undeclared identifier: Width in the script engine before any Try/Except ran. That stops the polling loop until a human dismisses the dialog and re-runs StartMCPServer, and every queued call fails meanwhile. Seen on a live board on 2026-09-14 while exporting coil copper (tracks + arcs) for a simulation.

One-line fix: the getter reads Arc.LineWidth.

🤖 Generated with Claude Code

…getter

IPCB_Arc has no Width member, so obj_query with Width on an eArcObject raised a modal
'Undeclared identifier: Width' in the script engine before any Try/Except ran, which stopped
the polling loop until a human dismissed it and re-ran StartMCPServer. Reported from a live
board on 2026-09-14 while exporting coil copper.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@salitronic

Copy link
Copy Markdown
Owner

Merged as 2d1630b.

Checked for other sites before taking it: PCB.pas:10200 already reads Arc.LineWidth, so PCBGeneric.pas:166 was the last one left. Thanks for tracking it down from a wedged loop, that class of fault is miserable to diagnose.

Cherry-picked onto main rather than merged through the PR, so GitHub will show this as closed rather than merged. The commit carries your authorship.

@salitronic salitronic closed this Sep 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants