Ignore return value of function declared with 'warn_unused_result'

David Eklov · facebook-github-bot · commit 1a5b3ab336f2 · 2022-08-31T13:13:49.000-07:00
Summary:
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());

Differential Revision: D39181069

fbshipit-source-id: e9516178b645729874a7cc1b82720789a1aca0d1
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() {`