search

embeddedartistry / libc

libc targeted for embedded systems usage. Reduced set of functionality (due to embedded nature). Chosen for portability and quick bringup.
MIT License
510 stars 67 forks source link

Add inttypes.h #109

Closed patstew closed 5 years ago

patstew commented 5 years ago

I've added inttypes.h, it's basically nicked from musl. I've moved imax{abs/div} from stdlib.h. I've added the 'h' specifiers in, because the printf library seems to respect them, but they could be removed to make format strings slightly shorter.

phillipjohnston commented 5 years ago

Thanks for this PR, looks great. Just need to figure out why my CI pipeline didn't run for your branch. :)

phillipjohnston commented 5 years ago

Your changes uncovered some test errors - I will merge them in a separate PR.

phillipjohnston commented 5 years ago

Because your commit uncovered other problems, I needed to make additional changes. I think it's easier for me to do that rather than have you make the changes on your end, or for me to submit a PR to you which then gets merged back to me. I've incorporated your changes in #110 and given you credit with the commit.