Skip to content
This repository has been archived by the owner on Jan 2, 2021. It is now read-only.

missing-prototypes #312

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

missing-prototypes #312

wants to merge 3 commits into from

Commits on Jan 3, 2020

  1. missing-prototypes 

    Contributor @strager had recommended enabling missing-prototypes, which proved us eful for identifying unused code. In addition to missing-prototypes, this will en able -Wall and -Werror for C code.
    @NaruFGT
    NaruFGT committed Jan 3, 2020
    Configuration menu
    Copy the full SHA
    45a1357 View commit details
    Browse the repository at this point in the history

Commits on Jan 9, 2020

  1. Revert "missing-prototypes" 

    add_compile_options() will need CMake generator expressions to distinguish between invocations of C and C++ compilers. This commit did not solve the issue and may cause build errors if the missing-prototypes option is passed to C++ compilers.

This reverts commit 45a1357.
    @NaruFGT
    NaruFGT committed Jan 9, 2020
    Configuration menu
    Copy the full SHA
    a78cdea View commit details
    Browse the repository at this point in the history

  2. missing-prototypes & missing-declarations 

    Contributor @strager had recommended enabling missing-prototypes, which proved useful for identifying unused code.
I have added warning options -Wmissing-declarations and for the C compiler -Wmissing-prototypes.
Until these warnings are taken care of, I've disabled errors for these warnings.
    @NaruFGT
    NaruFGT committed Jan 9, 2020
    Configuration menu
    Copy the full SHA
    32e37e9 View commit details
    Browse the repository at this point in the history