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

refactor: eliminate many of the compiler warnings. #1858

Merged
merged 5 commits into from
Nov 2, 2022
Merged

refactor: eliminate many of the compiler warnings. #1858

merged 5 commits into from
Nov 2, 2022

Commits on Nov 1, 2022

  1. refactor: eliminate many of the compiler warnings.

    @jpivarski
    jpivarski committed Nov 1, 2022
    Configuration menu
    Copy the full SHA
    9ac6e2c View commit details
    Browse the repository at this point in the history

  2. Fix warnings on MacOS, too. (All but one.)

    @jpivarski
    jpivarski committed Nov 1, 2022
    Configuration menu
    Copy the full SHA
    9a82c04 View commit details
    Browse the repository at this point in the history

  3. Maybe this will fix the warning on Windows.

    @jpivarski
    jpivarski committed Nov 1, 2022
    Configuration menu
    Copy the full SHA
    f2aba3e View commit details
    Browse the repository at this point in the history

  4. Explicit cast as an ancient C type to squash complaints about printf … 

    …format.
    @jpivarski
    jpivarski committed Nov 1, 2022
    Configuration menu
    Copy the full SHA
    c65ee0b View commit details
    Browse the repository at this point in the history

Commits on Nov 2, 2022

  1. Reintroduce unused argument names as comments, and some with FIXMEs.

    @jpivarski
    jpivarski committed Nov 2, 2022
    Configuration menu
    Copy the full SHA
    b6bfaf4 View commit details
    Browse the repository at this point in the history