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

Include all SntpStatus_t values in Sntp_StatusToStr #83

Merged
merged 3 commits into from
Jan 7, 2024
Merged

Include all SntpStatus_t values in Sntp_StatusToStr #83

merged 3 commits into from
Jan 7, 2024

Commits on Jan 5, 2024

  1. Include all SntpStatus_t values in Sntp_StatusToStr 

    Withtout this change, the function Sntp_StatusToStr would return
"Invalid status code!" for the following valid status codes:

- SntpRejectedResponse
- SntpErrorSendTimeout
- SntpErrorResponseTimeout
- SntpNoResponseReceived
- SntpErrorContextNotInitialized

This issue was reported here - FreeRTOS#82

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
    @aggarg
    aggarg committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    41e2236 View commit details
    Browse the repository at this point in the history

  2. Fix memory estimates 

    Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
    @aggarg
    aggarg committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    e1a1963 View commit details
    Browse the repository at this point in the history

  3. Fix spell check 

    Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
    @aggarg
    aggarg committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    d8f428f View commit details
    Browse the repository at this point in the history