Skip to content

feat(os): add os_snprintf definition - #436

Merged
aloisklink merged 1 commit into
mainfrom
feat/add-os-snprintf
Feb 21, 2023
Merged

feat(os): add os_snprintf definition#436
aloisklink merged 1 commit into
mainfrom
feat/add-os-snprintf

Conversation

@aloisklink

Copy link
Copy Markdown
Contributor

Source-code taken from the hostap project expects a os_snprintf function/macro that points to the ISO C99 snprintf.


Adapted from commit dff384c.

I've instead given it some arguments to make it clear that it's a function.

Source-code taken from the hostap project expects a `os_snprintf`
function/macro that points to the ISO C99 `snprintf`.

Co-authored-by: Alexandru Mereacre <mereacre@gmail.com>
@aloisklink aloisklink added the enhancement New feature or request label Feb 21, 2023
@aloisklink aloisklink added this to the RADIUS ERP Server milestone Feb 21, 2023
@mereacre
mereacre self-requested a review February 21, 2023 16:14
@aloisklink
aloisklink enabled auto-merge February 21, 2023 16:15
@aloisklink
aloisklink added this pull request to the merge queue Feb 21, 2023
@codecov

codecov Bot commented Feb 21, 2023

Copy link
Copy Markdown

Codecov Report

Merging #436 (6d0a5b6) into main (595fe95) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #436   +/-   ##
=======================================
  Coverage   53.08%   53.08%           
=======================================
  Files         144      144           
  Lines       19870    19870           
=======================================
  Hits        10547    10547           
  Misses       9323     9323           
Impacted Files Coverage Δ
src/utils/os.h 100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Merged via the queue into main with commit 51b815f Feb 21, 2023
@aloisklink
aloisklink deleted the feat/add-os-snprintf branch February 21, 2023 16:32
@aloisklink aloisklink mentioned this pull request Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants