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

Unified Diff: lily/include/midi-chunk.hh

Issue 551320043: Issue 5650: Use C++11 "override" keyword (Closed)
Patch Set: Compile with -Werror=suggest-override, when supported Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lily/include/lily-lexer.hh ('k') | lily/include/midi-item.hh » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lily/include/midi-chunk.hh
diff --git a/lily/include/midi-chunk.hh b/lily/include/midi-chunk.hh
index e500cdb7de879d07e92ed4ec9921f5b1dc00fb72..6bff3b87328fe87a53dc4cfdeac0e47eecdee3cf 100644
--- a/lily/include/midi-chunk.hh
+++ b/lily/include/midi-chunk.hh
@@ -46,7 +46,7 @@ public:
void set (const string &header_string, const string &data_string, const string &footer_string);
virtual string to_string () const;
virtual string data_string () const;
- DECLARE_CLASSNAME (Midi_chunk);
+ VIRTUAL_CLASS_NAME (Midi_chunk);
virtual ~Midi_chunk ();
private:
string data_string_;
@@ -57,7 +57,7 @@ private:
class Midi_header : public Midi_chunk
{
public:
- DECLARE_CLASSNAME (Midi_header);
+ OVERRIDE_CLASS_NAME (Midi_header);
Midi_header (int format, int tracks, int clocks_per_4);
};
@@ -67,7 +67,7 @@ class Midi_track : public Midi_chunk
public:
int number_;
int port_;
- DECLARE_CLASSNAME (Midi_track);
+ OVERRIDE_CLASS_NAME (Midi_track);
vector<Midi_event *> events_;
@@ -75,7 +75,7 @@ public:
~Midi_track ();
void add (int, Midi_item *midi);
- virtual string data_string () const;
+ string data_string () const override;
void push_back (int, Midi_item *midi);
};
« no previous file with comments | « lily/include/lily-lexer.hh ('k') | lily/include/midi-item.hh » ('j') | no next file with comments »

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