Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(844)

Issue 561790043: Remove define causing a warning

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months, 1 week ago by hahnjo
Modified:
2 months, 1 week ago
Reviewers:
Dan Eble, lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Remove define causing a warning These lines were added in commit 25beb8d01b in 2008 to provide compatibility with GCC 3.4 which clearly does not support C++11. (The warning is visible since commit b96b6334c0 which put system headers last, so that the define is already provided transitvely.)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -4 lines) Patch
M lily/source-file.cc View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 2
lemzwerg
LGTM
2 months, 1 week ago (2020-05-04 17:08:56 UTC) #1
Dan Eble
2 months, 1 week ago (2020-05-04 20:57:22 UTC) #2
Thank you.  This was bugging me.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b