summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rwxr-xr-xconfigure.py5
-rw-r--r--src/browse.cc2
-rw-r--r--src/build.cc2
-rw-r--r--src/build_test.cc2
-rw-r--r--src/disk_interface_test.cc6
-rw-r--r--src/graph.cc4
-rw-r--r--src/ninja.cc2
-rw-r--r--src/subprocess.cc2
-rw-r--r--src/test.cc2
9 files changed, 15 insertions, 12 deletions
diff --git a/configure.py b/configure.py
index fbcdeb3..ef4bec1 100755
--- a/configure.py
+++ b/configure.py
@@ -74,7 +74,10 @@ n.variable('builddir', 'build')
n.variable('cxx', os.environ.get('CXX', 'g++'))
n.variable('ar', os.environ.get('AR', 'ar'))
-cflags = ['-g', '-Wall', '-Wextra', '-Wno-deprecated', '-fno-exceptions',
+cflags = ['-g', '-Wall', '-Wextra',
+ '-Wno-deprecated',
+ '-Wno-unused-parameter',
+ '-fno-exceptions',
'-fvisibility=hidden', '-pipe']
if not options.debug:
cflags += ['-O2', '-DNDEBUG']
diff --git a/src/browse.cc b/src/browse.cc
index 2e74206..bab3f36 100644
--- a/src/browse.cc
+++ b/src/browse.cc
@@ -20,7 +20,7 @@
#include "../build/browse_py.h"
-void RunBrowsePython(State* /* state */, const char* ninja_command,
+void RunBrowsePython(State* state, const char* ninja_command,
const char* initial_target) {
// Fork off a Python process and have it run our code via its stdin.
// (Actually the Python process becomes the parent.)
diff --git a/src/build.cc b/src/build.cc
index 25d3ba9..bb73f7d 100644
--- a/src/build.cc
+++ b/src/build.cc
@@ -421,7 +421,7 @@ struct DryRunCommandRunner : public CommandRunner {
finished_.push(edge);
return true;
}
- virtual Edge* WaitForCommand(bool* success, string* /* output */) {
+ virtual Edge* WaitForCommand(bool* success, string* output) {
if (finished_.empty())
return NULL;
*success = true;
diff --git a/src/build_test.cc b/src/build_test.cc
index 6beb976..0d4dce6 100644
--- a/src/build_test.cc
+++ b/src/build_test.cc
@@ -251,7 +251,7 @@ bool BuildTest::StartCommand(Edge* edge) {
return true;
}
-Edge* BuildTest::WaitForCommand(bool* success, string* /* output */) {
+Edge* BuildTest::WaitForCommand(bool* success, string* output) {
if (Edge* edge = last_command_) {
if (edge->rule_->name_ == "fail")
*success = false;
diff --git a/src/disk_interface_test.cc b/src/disk_interface_test.cc
index 30fa4ca..107726b 100644
--- a/src/disk_interface_test.cc
+++ b/src/disk_interface_test.cc
@@ -163,15 +163,15 @@ struct StatTest : public StateTestWithBuiltinRules,
public DiskInterface {
// DiskInterface implementation.
virtual int Stat(const string& path);
- virtual bool MakeDir(const string& /* path */) {
+ virtual bool MakeDir(const string& path) {
assert(false);
return false;
}
- virtual string ReadFile(const string& /* path */, string* /* err */) {
+ virtual string ReadFile(const string& path, string* err) {
assert(false);
return "";
}
- virtual int RemoveFile(const string& /* path */) {
+ virtual int RemoveFile(const string& path) {
assert(false);
return 0;
}
diff --git a/src/graph.cc b/src/graph.cc
index 2cdba80..d881280 100644
--- a/src/graph.cc
+++ b/src/graph.cc
@@ -203,8 +203,8 @@ bool Edge::LoadDepFile(State* state, DiskInterface* disk_interface,
// Check that this depfile matches our output.
StringPiece opath = StringPiece(outputs_[0]->file_->path_);
if (opath != makefile.out_) {
- *err = "expected makefile to mention '" + outputs_[0]->file_->path_ + "', "
- "got '" + makefile.out_.AsString() + "'";
+ *err = "expected depfile '" + path + "' to mention '" +
+ outputs_[0]->file_->path_ + "', got '" + makefile.out_.AsString() + "'";
return false;
}
diff --git a/src/ninja.cc b/src/ninja.cc
index 8125c98..a46980e 100644
--- a/src/ninja.cc
+++ b/src/ninja.cc
@@ -367,7 +367,7 @@ int CmdTargets(State* state, int argc, char* argv[]) {
}
}
-int CmdRules(State* state, int /* argc */, char* /* argv */[]) {
+int CmdRules(State* state, int argc, char* /* argv */[]) {
for (map<string, const Rule*>::iterator i = state->rules_.begin();
i != state->rules_.end(); ++i) {
if (i->second->description_.unparsed_.empty()) {
diff --git a/src/subprocess.cc b/src/subprocess.cc
index da93b1f..74eded0 100644
--- a/src/subprocess.cc
+++ b/src/subprocess.cc
@@ -37,7 +37,7 @@ Subprocess::~Subprocess() {
Finish();
}
-bool Subprocess::Start(SubprocessSet* /* set */, const string& command) {
+bool Subprocess::Start(SubprocessSet* set, const string& command) {
int output_pipe[2];
if (pipe(output_pipe) < 0)
Fatal("pipe: %s", strerror(errno));
diff --git a/src/test.cc b/src/test.cc
index 24b68f6..719cec3 100644
--- a/src/test.cc
+++ b/src/test.cc
@@ -51,7 +51,7 @@ bool VirtualFileSystem::MakeDir(const string& path) {
return true; // success
}
-string VirtualFileSystem::ReadFile(const string& path, string* /* err */) {
+string VirtualFileSystem::ReadFile(const string& path, string* err) {
files_read_.push_back(path);
FileMap::iterator i = files_.find(path);
if (i != files_.end())