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

Issue 561790043: Remove define causing a warning (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 years, 11 months ago by hahnjo
Modified:
3 years, 5 months 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
3 years, 11 months ago (2020-05-04 17:08:56 UTC) #1
Dan Eble
3 years, 11 months 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