diff options
author | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2017-12-12T10·47+0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-12-12T10·47+0100 |
commit | de38fa9987fcd1c2b79c826d49bd235dbd241676 (patch) | |
tree | 94c5d433aa95003ed02a566a0f78d48e5d86d1cb | |
parent | e3c19ff9bcbcaf2ec8e7c3d7f964e8d8aefaea43 (diff) | |
parent | 00e0c416ffccf6edb63fe7f4b01422915ff12f86 (diff) |
Merge pull request #1724 from dtzWill/fix/minor-warnings
fix minor warnings
-rw-r--r-- | src/libstore/build.cc | 4 | ||||
-rw-r--r-- | src/libutil/logging.cc | 1 | ||||
-rw-r--r-- | src/nix/command.hh | 2 |
3 files changed, 4 insertions, 3 deletions
diff --git a/src/libstore/build.cc b/src/libstore/build.cc index 866964a4c465..d4bd650baf22 100644 --- a/src/libstore/build.cc +++ b/src/libstore/build.cc @@ -79,7 +79,7 @@ typedef std::shared_ptr<Goal> GoalPtr; typedef std::weak_ptr<Goal> WeakGoalPtr; struct CompareGoalPtrs { - bool operator() (const GoalPtr & a, const GoalPtr & b); + bool operator() (const GoalPtr & a, const GoalPtr & b) const; }; /* Set of goals. */ @@ -178,7 +178,7 @@ protected: }; -bool CompareGoalPtrs::operator() (const GoalPtr & a, const GoalPtr & b) { +bool CompareGoalPtrs::operator() (const GoalPtr & a, const GoalPtr & b) const { string s1 = a->key(); string s2 = b->key(); return s1 < s2; diff --git a/src/libutil/logging.cc b/src/libutil/logging.cc index 011155871122..6924e0080475 100644 --- a/src/libutil/logging.cc +++ b/src/libutil/logging.cc @@ -49,6 +49,7 @@ public: void startActivity(ActivityId act, Verbosity lvl, ActivityType type, const std::string & s, const Fields & fields, ActivityId parent) + override { if (lvl <= verbosity && !s.empty()) log(lvl, s + "..."); diff --git a/src/nix/command.hh b/src/nix/command.hh index daa3b3fa7030..6b34e3881e79 100644 --- a/src/nix/command.hh +++ b/src/nix/command.hh @@ -6,7 +6,7 @@ namespace nix { struct Value; -struct Bindings; +class Bindings; class EvalState; /* A command is an argument parser that can be executed by calling its |