IMPALA-4535: Remove 'auto' from parameter list

Although GCC supports void f(auto arg) { }, this isn't part of C++14.

Testing: Local ASAN build passed.

Change-Id: I838aa2c1081f7ed21cc2faa209a9eaa4fd0512a4
Reviewed-on: http://gerrit.cloudera.org:8080/5214
Reviewed-by: Tim Armstrong <[email protected]>
Tested-by: Internal Jenkins


Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/a35eb2fe
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/a35eb2fe
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/a35eb2fe

Branch: refs/heads/master
Commit: a35eb2fedb0ea565c766f4dd55bab2ca866f5cb2
Parents: e4fc5bd
Author: Henry Robinson <[email protected]>
Authored: Wed Nov 23 17:45:05 2016 -0800
Committer: Internal Jenkins <[email protected]>
Committed: Thu Nov 24 06:14:20 2016 +0000

----------------------------------------------------------------------
 be/src/util/scope-exit-trigger.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/a35eb2fe/be/src/util/scope-exit-trigger.h
----------------------------------------------------------------------
diff --git a/be/src/util/scope-exit-trigger.h b/be/src/util/scope-exit-trigger.h
index 42b808e..bc2b702 100644
--- a/be/src/util/scope-exit-trigger.h
+++ b/be/src/util/scope-exit-trigger.h
@@ -28,7 +28,7 @@ namespace impala {
 /// act as proxy for work-performing d'tors, which we try to avoid.
 class ScopeExitTrigger {
  public:
-  ScopeExitTrigger(const auto& trigger) : trigger_(trigger) {}
+  ScopeExitTrigger(const std::function<void()>& trigger) : trigger_(trigger) {}
 
   ~ScopeExitTrigger() { trigger_(); }
 

Reply via email to