#251 New Compiler Warning Flag


I've added two new compiler warning flags: Wunreachable-code and Wstrict-prototypes. The patch includes the new flags and some changes I made to reduce the number of warnings produced (all by Wstrict-prototypes; Wunreachable-code does not complain at all). Also attached is the output from make -k.

The warnings I couldn't fix were generated for prototypes that had variable-argument functions as arguments. I've tried to fix all of the rest that I could!

2 Attachments


  • Anonymous - 2014-01-02

    In dringenden Fällen kontaktieren Sie mich ab 15:00 unter 079 422 6653
    In allen anderen Fällen wenden Sie sich bite an René Stanger, Panagiotis Pantazis oder Enrico Moor.

    Vielen Dank und eine gute Woche.
    Josef Stadelmann

  • shardulc

    shardulc - 2014-01-02

    Corrected build log and added only the Wstrict-prototypes flag. Changed patch likewise -- please see attachment.

  • shardulc

    shardulc - 2014-01-02

    I managed to fix almost all of the warnings -- all except one, which I just can't make sense of. The resulting build log and patch file is attached.

  • shardulc

    shardulc - 2014-01-02

    Fixed the one I couldn't fix as well. The pointers are complicated, needs more documentation.

  • shardulc

    shardulc - 2014-01-03

    The patch attached fixes the errors without the extra Wstrict-prototypes flag, whose causes were my own edits. The output of make -k VERBOSE=1, including stderr, is given in build_without.log.
    The patch also fixes some warnings with the Wstrict-prototypes flag; the rest are given in build_with.log.

  • shardulc

    shardulc - 2014-01-04

    Attached patch contains a few more fixes, which build cleanly without the extra flag (see build_without.log) and fix some of the errors with the flag (for the others, see build_with.log). The patch is meant to be applied to the latest revision and not the earlier one.

  • Sean Morrison

    Sean Morrison - 2014-01-04
    • status: open --> open-accepted
    • assigned_to: Sean Morrison

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

No, thanks