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.

Sign up for GitHub

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

Jump to bottom

Add strtoul, strcpy #7

Merged
merged 3 commits into from
Aug 17, 2022
Merged

Add strtoul, strcpy #7

merged 3 commits into from
Aug 17, 2022

Conversation

sawi97
Copy link
Contributor

@sawi97 sawi97 commented Aug 16, 2022 •
edited
Loading

This will make it possible to link with nrfxlib v2.0x.

Added:

  • strcpy
  • strtoul

@sawi97 sawi97 changed the title Add strtoul Add strtoul, strcpy Aug 16, 2022
eldruin
eldruin reviewed Aug 17, 2022
View reviewed changes
Copy link
Member

@eldruin eldruin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!
Could you add an entry to the changelog?

@sawi97
Copy link
Contributor Author

sawi97 commented Aug 17, 2022

Of course, like this?

eldruin
eldruin approved these changes Aug 17, 2022
View reviewed changes
Copy link
Member

@eldruin eldruin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@eldruin eldruin merged commit d356185 into rust-embedded-community:master Aug 17, 2022
@sawi97 sawi97 deleted the strtoul branch March 15, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eldruin eldruin eldruin 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.
2 participants
@sawi97 @eldruin