Skip to content
Jump to bottom

Fix warnings #4

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

Fix warnings #4

wants to merge 4 commits into from

Commits on Oct 7, 2017

  1. param.cpp: Remove \r at end of line 

    This file had some lines terminated with "\r\n" and
others with "\n" only, which most editors don't like.

Convert to "\n" line endings.
    @donald
    donald committed Oct 7, 2017
    Configuration menu
    Copy the full SHA
    40bb785 View commit details
    Browse the repository at this point in the history

  2. param.c: Remove dangling whitespace

    @donald
    donald committed Oct 7, 2017
    Configuration menu
    Copy the full SHA
    35c2204 View commit details
    Browse the repository at this point in the history

  3. param.cpp: Avoid narrowing conversation warnings

    @donald
    donald committed Oct 7, 2017
    Configuration menu
    Copy the full SHA
    59a9872 View commit details
    Browse the repository at this point in the history

  4. Fix more warnings 

    No functional change intended.

Various fixes to prevent warnings.

In fact, we are clean now for

    make CXXFLAGS="-Wall -Werror"

But only because "-Wall" will not be propagated
to the subdirectories...
    @donald
    donald committed Oct 7, 2017
    Configuration menu
    Copy the full SHA
    a7bd5a7 View commit details
    Browse the repository at this point in the history