Use ARROW_UNUSED for ignored results

alinaliBQ · alinaliBQ · commit 34281ec1825b · 2025-10-22T13:44:18.000-07:00
diff --git a/cpp/src/arrow/flight/sql/odbc/odbc_api.cc b/cpp/src/arrow/flight/sql/odbc/odbc_api.cc
@@ -423,7 +423,8 @@ SQLRETURN SQLGetDiagRec(SQLSMALLINT handle_type, SQLHANDLE handle, SQLSMALLINT r
     // Microsoft documentation does not mention
     // any SQLGetDiagRec return value that is associated with `sql_state` buffer, so
     // the return value for writing `sql_state` buffer is ignored by the driver.
-    GetStringAttribute(is_unicode, state, false, sql_state, size, &size, *diagnostics);
+    ARROW_UNUSED(GetStringAttribute(is_unicode, state, false, sql_state, size, &size,
+                                    *diagnostics));
   }
 
   if (native_error_ptr) {

Original file line number	Diff line number	Diff line change
`@@ -423,7 +423,8 @@ SQLRETURN SQLGetDiagRec(SQLSMALLINT handle_type, SQLHANDLE handle, SQLSMALLINT r`
`423`	`423`	`// Microsoft documentation does not mention`
`424`	`424`	// any SQLGetDiagRec return value that is associated with `sql_state` buffer, so
`425`	`425`	// the return value for writing `sql_state` buffer is ignored by the driver.
`426`		`- GetStringAttribute(is_unicode, state, false, sql_state, size, &size, *diagnostics);`
	`426`	`+ ARROW_UNUSED(GetStringAttribute(is_unicode, state, false, sql_state, size, &size,`
	`427`	`+ *diagnostics));`
`427`	`428`	`}`
`428`	`429`
`429`	`430`	`if (native_error_ptr) {`