Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Regression: Fix issue with argdecode #1878

Closed
kunstmusik opened this issue May 12, 2024 · 1 comment
Closed

Regression: Fix issue with argdecode #1878

kunstmusik opened this issue May 12, 2024 · 1 comment
Assignees
@kunstmusik @vlazzarini
Labels
bug Csound7

Comments

@kunstmusik
Copy link
Member

argdecode was modified to fix an issue in #1859 with setOption failures calling longjmp. This however caused issues with commandline csound not exiting early as it should on a failed argument. Diagnosing this, argdecode should be rewritten to use return values and calling code updated to use the return values. (See comment in https://github.com/csound/csound/pull/1859/files#r1596198649).
@kunstmusik kunstmusik mentioned this issue May 12, 2024
Merged
vlazzarini pushed a commit that referenced this issue May 18, 2024
fixing #1878
7af7831
@vlazzarini
Copy link
Member

Fixed in #1885.

kunstmusik added a commit that referenced this issue Jun 1, 2024
@kunstmusik
Merge pull request #1885 from csound/feature/argdecode
685d0ce 
fixing #1878
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kunstmusik kunstmusik

@vlazzarini vlazzarini

Labels
bug Csound7
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kunstmusik @vlazzarini