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

Bugfix: scanf and fscanf argument count mismatch #96

Merged
merged 10 commits into from
May 29, 2022

Conversation

boesing
Copy link
Owner

@boesing boesing commented May 29, 2022

In case that 0 arguments are passed to consume the matched template parmeters, sscanf and fscanf do return the matched variables instead of returning them via reference.

fixes #23

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
…te classes

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
…e matches

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing added the bug Something isn't working label May 29, 2022
@boesing boesing added this to the 1.2.0 milestone May 29, 2022
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Due to the usage of psalm, there is absolutely no need for a codestyle check - which can only interact with a single scoped file - to tell developers that some type-hint might be missing.

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing merged commit d70603e into 1.2.x May 29, 2022
@boesing boesing deleted the bugfix/scanf-argument-count-mismatch branch May 29, 2022 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sscanf is not properly handled
1 participant