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

feat common: slight code improvement #9

Closed
wants to merge 3 commits into from
Closed

feat common: slight code improvement #9

wants to merge 3 commits into from

Commits on Jun 17, 2024

  1. cmake: build tests 

    To check when modifying code when using сmake, it is worth collecting all binary files.
The code is compiled only with gnu extensions of the standard. This should be explicitly stated.

Signed-off-by: Aleksandr Golubev <[email protected]>
    Aleksandr Golubev committed Jun 17, 2024
    Configuration menu
    Copy the full SHA
    f30b414 View commit details
    Browse the repository at this point in the history

  2. tests: use a thread-safe function to transform time 

    The test logging contains a call to a not-thread-save method localtime.

Signed-off-by: Aleksandr Golubev <[email protected]>
    Aleksandr Golubev committed Jun 17, 2024
    Configuration menu
    Copy the full SHA
    1cc984a View commit details
    Browse the repository at this point in the history

  3. build: fixing unused-result warning 

    The results of calls to methods write() and read() are now checked in the debug build, which will help diagnose potential problems.

Signed-off-by: Aleksandr Golubev <[email protected]>
    Aleksandr Golubev committed Jun 17, 2024
    Configuration menu
    Copy the full SHA
    2cee9b2 View commit details
    Browse the repository at this point in the history