Hello Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4773
to review the following change.
Change subject: tool_action_test.cc: specify lambda return type
......................................................................
tool_action_test.cc: specify lambda return type
A user reported that they were unable to compile tool_action_test.cc on
OS X 10.11 with either the system Clang or thirdparty Clang. It appears
that these toolchains are not able to infer the correct return type for
a lambda. I'm not able to repro the failure, but the original user
confirmed that this patch fixed the issue. Failure:
/Users/<user>/ProjectFiles/kudu/src/kudu/tools/tool_action_test.cc:384:5:
error: return type 'kudu::Status' must match previous return type 'const
::kudu::Status' when lambda
expression has unspecified explicit return type
return Status::OK();
^
1 error generated.
Change-Id: I52824e85a37c67bea8a2373264e8198d8d2294f4
---
M src/kudu/tools/tool_action_test.cc
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/4773/1
--
To view, visit http://gerrit.cloudera.org:8080/4773
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I52824e85a37c67bea8a2373264e8198d8d2294f4
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
