Skip to content

Commit

Permalink
Merge pull request #13436 from Liuchang0812/cleanup-mgr
Browse files Browse the repository at this point in the history
mgr: add override in mgr subsystem

Reviewed-by: Sage Weil <sage@redhat.com>
  • Loading branch information
liewegas committed Feb 15, 2017
2 parents efbc836 + 491d653 commit 7058133
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/mgr/Mgr.cc
Expand Up @@ -77,7 +77,7 @@ class MetadataUpdate : public Context
MetadataUpdate(DaemonStateIndex &daemon_state_, const DaemonKey &key_)
: daemon_state(daemon_state_), key(key_) {}

void finish(int r)
void finish(int r) override
{
daemon_state.clear_updating(key);
if (r == 0) {
Expand Down
2 changes: 1 addition & 1 deletion src/mgr/PyModules.cc
Expand Up @@ -399,7 +399,7 @@ class ServeThread : public Thread
ServeThread(MgrPyModule *mod_)
: mod(mod_) {}

void *entry()
void *entry() override
{
PyGILState_STATE gstate;
gstate = PyGILState_Ensure();
Expand Down
2 changes: 1 addition & 1 deletion src/mgr/PyState.cc
Expand Up @@ -48,7 +48,7 @@ class MonCommandCompletion : public Context
Py_DECREF(python_completion);
}

void finish(int r)
void finish(int r) override
{
PyGILState_STATE gstate;
gstate = PyGILState_Ensure();
Expand Down

0 comments on commit 7058133

Please sign in to comment.