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

core: ustrtoull: convert zero length strings to 0 #39

Conversation

rettichschnidi
Copy link
Contributor

@rettichschnidi rettichschnidi commented Sep 4, 2018

Without this change, parse_common_attributes() can not deal with an empty offset value.

strtoull sets errno to EINVAL for empty strings.

This change mimics the behavior as it was before the ustrtoull
refactoring in 6e4f913.
@sbabic
Copy link
Owner

sbabic commented Sep 4, 2018

It can be, but....

  • github is not used for PR
  • please send a patch to SWUpdate's ML according to the contribution page in doc.

Thanks !

@sbabic sbabic closed this Sep 4, 2018
@gmacario
Copy link

gmacario commented Sep 4, 2018

Hello @sbabic,

github is not used for PR

May I then suggest to go to repository Settings and disable the "Pulls" menu?

That will help reduce both contributors and maintainers work.

@sbabic
Copy link
Owner

sbabic commented Sep 4, 2018 via email

@gmacario
Copy link

gmacario commented Sep 4, 2018

Hi @sbabic, my apologies I wrote my previous reply on the phone, with the assumption that the setting was close to the one for the Issues.
Tonight when back at my desktop I realized you are absolutely right, and apparently you are not alone requesting this feature:

Sorry for the noise, and thanks again for your work with swupdate!

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.

3 participants