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

Fix almost every warning #2195

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix almost every warning #2195

wants to merge 4 commits into from

Commits on Nov 11, 2024

  1. Fix almost every warning as of Clang 19 

    * <codecvt> is deprecated, we can use QString's UTF-16 conversion
  instead
* the sem_timedwait implementation had warnings in it, but I just
  removed it entirely because we don't use it anymore anyway
    @nadiaholmquist
    nadiaholmquist committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    66cdc3d View commit details
    Browse the repository at this point in the history

  2. I'm not 100% sure about this one, but unless there's some weird C++ shit 

    going on here it should be fine
    @nadiaholmquist
    nadiaholmquist committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    95c5f33 View commit details
    Browse the repository at this point in the history

  3. don't unconditionally redefine CLOCK_MONOTONIC on Windows, also set the 

    redefinition to the actual value it's shadowing
    @nadiaholmquist
    nadiaholmquist committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    e0475b0 View commit details
    Browse the repository at this point in the history

  4. change usages of deprecated sprintf to snprintf

    @nadiaholmquist
    nadiaholmquist committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    b7e933a View commit details
    Browse the repository at this point in the history