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.

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

fixed Internal method as published by FindNonVoid #182

Closed
wants to merge 1 commit into from

Conversation

flydev-fr
Copy link
Sponsor Contributor

  • In new command-line parser a0d1ec6, FindNonVoidRawUtf8 & FindNonVoidRawUtf8I trigger an access violation when only the switch symbol is given as argument, case-sensitive or not.

- In new command-line parser a0d1ec6, `FindNonVoidRawUtf8` & `FindNonVoidRawUtf8I` trigger an access violation when only the switch symbol is given as argument, case-sensitive or not.
@synopse
Copy link
Owner

synopse commented May 1, 2023

This is not the proper fix.
You change the behavior of a function which is named FindNonVoidRawUtf8
The caller should be fixed, because the function is EXPECTED to have non-void values.

Please use the forum for such discussion.

@synopse synopse closed this May 1, 2023
@synopse
Copy link
Owner

synopse commented May 1, 2023

I suspect we did make a fix on last Friday, which may be related to your issue.
554ff23

Please try with the latest version of the framework.

@synopse
Copy link
Owner

synopse commented May 2, 2023

After investigation, it seems that last week fix was not enough.

Please try with 38f27f08

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.

None yet

2 participants