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

Fix fscanf execute order when reading ASCII format. #63

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

kimidaisuki22
Copy link

Keep consist of order of evaluation in after optimum on some compilers, such as MSVC.

@kimidaisuki22
Copy link
Author

Note: When using MSVC with visual studio 2019 and build it with release version, it executes fscanf out of order and leads the reading to fail.

@hroncok
Copy link
Member

hroncok commented Dec 15, 2022

I see. Can you please not combine declarations and code?

@kimidaisuki22
Copy link
Author

I see. Can you please not combine declarations and code?

Sure, I am glad to do that. Thanks.

src/stlinit.c Outdated Show resolved Hide resolved
src/stlinit.c Outdated Show resolved Hide resolved
@hroncok hroncok self-assigned this Dec 16, 2022
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