Ignore return value of function declared with 'warn_unused_result' (#84369)

David Eklov · facebook-github-bot · commit cd9cb59e6aa0 · 2022-09-07T10:59:48.000-07:00
Summary: X-link: pytorch/pytorch#84369 Pull Request resolved: #152 Ignore return value of function declared with 'warn_unused_result' Addresses the following build failure that we get on some of our internal build environments: caffe2/torch/csrc/deploy/environment.h:60:5: error: ignoring return value of function declared with 'warn_unused_result' attribute [-Werror,-Wunused-result] system(rmCmd.c_str()); Reviewed By: jiyuanzFB, PaliC Differential Revision: D39181069 fbshipit-source-id: 6b4daeba829d074f2a217e34daf97bbc4dd8afd4
diff --git a/multipy/runtime/environment.h b/multipy/runtime/environment.h
@@ -66,7 +66,7 @@ class Environment {
   }
   virtual ~Environment() {
     auto rmCmd = "rm -rf " + extraPythonLibrariesDir_;
-    system(rmCmd.c_str());
+    (void)system(rmCmd.c_str());
   }
   virtual void configureInterpreter(Interpreter* interp) = 0;
   virtual const std::vector<std::string>& getExtraPythonPaths() {

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ class Environment {`
`66`	`66`	`}`
`67`	`67`	`virtual ~Environment() {`
`68`	`68`	`auto rmCmd = "rm -rf " + extraPythonLibrariesDir_;`
`69`		`- system(rmCmd.c_str());`
	`69`	`+ (void)system(rmCmd.c_str());`
`70`	`70`	`}`
`71`	`71`	`virtual void configureInterpreter(Interpreter* interp) = 0;`
`72`	`72`	`virtual const std::vector<std::string>& getExtraPythonPaths() {`