Skip to content

Commit

Permalink
[codemod] Remove unused-variable in /fbcode/github/presto-trunk/prest…
Browse files Browse the repository at this point in the history
…o-native-execution/presto_cpp/main/QueryContextManager.cpp

Summary:
[codemod] Remove unused-variable in {filename}


LLVM-15 has a warning `-Wunused-variable` which we treat as an error because it's so often diagnostic of a code issue. Unused variables can compromise readability or, worse, performance.

This diff either (a) removes an unused variable and, possibly, it's associated code or (b) qualifies the variable with `[[maybe_unused]]`.

#buildsonlynotests - Builds are sufficient to verify
 - If you approve of this diff, please use the "Accept & Ship" button :-)

Differential Revision: D65445391
  • Loading branch information
r-barnes authored and amitkdutta committed Nov 7, 2024
1 parent 91f140c commit ef142c6
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -98,8 +98,6 @@ void updateVeloxConnectorConfigs(
std::unordered_map<
std::string,
std::unordered_map<std::string, std::string>>& connectorConfigStrings) {
const auto& systemConfig = SystemConfig::instance();

for (auto& entry : connectorConfigStrings) {
auto& connectorConfig = entry.second;

Expand Down

0 comments on commit ef142c6

Please sign in to comment.