Skip to content

assign outputs to varargout [#898]#899

Merged
4gwe merged 1 commit intodevelopfrom
898_data_editor_varargout
Apr 30, 2026
Merged

assign outputs to varargout [#898]#899
4gwe merged 1 commit intodevelopfrom
898_data_editor_varargout

Conversation

@gjheij
Copy link
Copy Markdown
Contributor

@gjheij gjheij commented Apr 29, 2026

Fixes #898.

This PR resolves errors occurring when closing pspm_data_editor, specifically:

Error: One or more output arguments not assigned during call to "varargout".
  • OutputFcn now assigns outputs based on nargout
  • Prevents errors when callers request more outputs than expected
  • Avoids fragile hard-coded assignments

@gjheij gjheij requested a review from 4gwe April 29, 2026 10:49
@4gwe 4gwe merged commit 0fe134e into develop Apr 30, 2026
1 check passed
@4gwe 4gwe deleted the 898_data_editor_varargout branch April 30, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@4gwe 4gwe 4gwe approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Warning when closing pspm_data_editor

2 participants

@gjheij @4gwe