Skip to content

Commit

Permalink
fixed (hopefully) warn_unused_result
Browse files Browse the repository at this point in the history
  • Loading branch information
brancomat committed Jan 7, 2016
1 parent 348505a commit bf20a28
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/arki-benchmark.cc
Original file line number Diff line number Diff line change
Expand Up @@ -336,7 +336,11 @@ int main(int argc, const char* argv[])
// Delete the dataset if it exists
// FIXME: unsafe, and not needed if --list is used, but we're a
// benchmark, not a general purpose tool
(void)system(("rm -rf " + i->second->value("path")).c_str());
int systemRet = system(("rm -rf " + i->second->value("path")).c_str());
if(systemRet == -1){
// FIXME: ponghino to get rid of warn_unused_result, see:
// http://stackoverflow.com/questions/9150397/warning-ignoring-return-value-of-system-c
}

// (Re)create the dataset from scratch
Benchmark::root()->addChild(new DSBenchmark(i->first, *i->second, info));
Expand Down

0 comments on commit bf20a28

Please sign in to comment.