Skip to content

EvaluteRTE: Correctly report AbortOnValue aborts with negative abort codes#500

Open
t-b wants to merge 2 commits into
mainfrom
bugfix/ignored-negative-abort-on-value-values
Open

EvaluteRTE: Correctly report AbortOnValue aborts with negative abort codes#500
t-b wants to merge 2 commits into
mainfrom
bugfix/ignored-negative-abort-on-value-values

Conversation

@t-b
Copy link
Copy Markdown
Member

@t-b t-b commented Jul 23, 2025

We need to assume that all error codes not being -1, -2, -3, -4 are from AbortOnValue as we can't differentiate them. This now correctly reports AbortOnValue aborts with negative abort codes.

Broken since a7ab54c (More detailed RTE error message, 2017-03-07). It did not help that the Igor Pro documentation is also wrong in this regard.

@t-b t-b added this to the Version 1.11 milestone Jul 23, 2025
@t-b t-b self-assigned this Jul 23, 2025
Comment thread procedures/igortest-basics.ipf
@t-b t-b force-pushed the bugfix/ignored-negative-abort-on-value-values branch 2 times, most recently from ebcc83c to 5d3d41a Compare August 31, 2025 21:24
@t-b t-b requested a review from Garados007 August 31, 2025 21:25
@t-b t-b force-pushed the bugfix/ignored-negative-abort-on-value-values branch from 5d3d41a to 090eb74 Compare September 1, 2025 14:51
…codes

We need to assume that all error codes not being -1, -2, -3, -4 are from
AbortOnValue as we can't differentiate them. This now correctly reports
AbortOnValue aborts with negative abort codes.

Broken since a7ab54c (More detailed RTE error message, 2017-03-07). It did
not help that the Igor Pro documentation is also wrong in this regard.
@t-b t-b force-pushed the bugfix/ignored-negative-abort-on-value-values branch from 090eb74 to 4594005 Compare January 23, 2026 17:15
The parameter is gone since 2f80d1a (Refactor TAP to use test result
waves, 2022-12-02).
@t-b t-b force-pushed the bugfix/ignored-negative-abort-on-value-values branch from 1415bd3 to d475ec7 Compare May 19, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants