lilypond-auto
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5608 Avoid ea

From: Auto mailings of changes to Lily Issues via Testlilyissues-auto
Subject: [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5608 Avoid easy warnings with Clang
Date: Mon, 25 Nov 2019 18:30:27 -0000

Pushed to staging as:

commit 9926ad3e43ab99c7463ca646fd4321647a9d72da
Author: Jonas Hahnfeld <address@hidden>
Date:   Tue Nov 19 18:56:26 2019 +0100

    Issue 5608/2: Fix struct vs. class template for PQueue

    Clang issues a warning that this could be an issue for the
    "Windows C++ ABI". Apparently, that's not really causing problems
    as the mismatch was introduced by commit c8989177a in 1997...

commit 0b62028523d8d81cbc81b48ce10d14d9f680510c
Author: Jonas Hahnfeld <address@hidden>
Date:   Tue Nov 19 18:55:29 2019 +0100

    Issue 5608/1: Fix warning flag detection for Clang

    Clang requires -Werror=unknown-warning-option to actually throw an
    error if it doesn't know about a warning flag.

[issues:#5608] Avoid easy warnings with Clang

Status: Fixed
Labels: Fixed_2_21_0
Created: Tue Nov 19, 2019 06:53 PM UTC by Jonas Hahnfeld
Last Updated: Mon Nov 25, 2019 05:34 PM UTC
Owner: Jonas Hahnfeld

Avoid easy warnings with Clang

Building with Clang is currently a lot nosier than with GCC. This
represents a first step into the right direction by picking the
low-hanging fruits.

  1. Fix warning flag detection for Clang

Clang requires -Werror=unknown-warning-option to actually throw an
error if it doesn't know about a warning flag.

  1. Fix struct vs. class template for PQueue

Clang issues a warning that this could be an issue for the
"Windows C++ ABI". Apparently, that's not really causing problems
as the mismatch was introduced by commit c8989177a in 1997...

http://codereview.appspot.com/573270043

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto














reply via email to








[Prev in Thread]
Current Thread
[Next in Thread]