| commit | 73d6d674c7578a170d5374a0369a1a78e94cc26d | |
| author | Jay Edgar <jkedgar@fb.com> | |
| Sat, 10 Sep 2022 01:30:44 +0000 (9 18:30 -0700) | ||
| committer | Facebook GitHub Bot <facebook-github-bot@users.noreply.github.com> | |
| Sat, 10 Sep 2022 01:30:44 +0000 (9 18:30 -0700) | ||
| tree | d54f837dfa32c29c75c4d722e9d4ea2dc3ecccd1 | treesnapshot (tar.gz zip) |
| parent | a0eabd5c77df5d97966596b2137ff6cddcf111a2 | commitdiff |
Fix unused variable
Summary: For an overridden function in a derived class where the base version had an optional argument, I thought I needed to match it even though the base class doesn't access the variable. This caused warnings which are treated as errors in some build environments. It turns out I was wrong and the signatures don't have to match on the optional part. Remove the variable name to avoid a warning.
Reviewed By: ibrahimjirdeh
Differential Revision: D39398985
fbshipit-source-id: d13105bde5a8095d92e93e7d4b81f3fa999020d1
Summary: For an overridden function in a derived class where the base version had an optional argument, I thought I needed to match it even though the base class doesn't access the variable. This caused warnings which are treated as errors in some build environments. It turns out I was wrong and the signatures don't have to match on the optional part. Remove the variable name to avoid a warning.
Reviewed By: ibrahimjirdeh
Differential Revision: D39398985
fbshipit-source-id: d13105bde5a8095d92e93e7d4b81f3fa999020d1
| third-party/squangle/src/squangle/mysql_client/SyncMysqlClient.h | diffblobblamehistory |