Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/14899


Change subject: [util] a small clean-up on MAYBE_RETURN_FAILURE
......................................................................

[util] a small clean-up on MAYBE_RETURN_FAILURE

Updated MAYBE_RETURN_FAILURE macro to follow the 'do {...} while (0)'
pattern so static code analyzers doesn't output warnings on an
empty statements when encountering constructs like
MAYBE_RETURN_FAILURE(...);

This patch doesn't contain any functional modifications.

Change-Id: Ieb9e172b4f2e1cfa712be08b9527f50bd0fe09e5
---
M src/kudu/util/env_posix.cc
M src/kudu/util/fault_injection.h
2 files changed, 8 insertions(+), 9 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/14899/1
--
To view, visit http://gerrit.cloudera.org:8080/14899
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ieb9e172b4f2e1cfa712be08b9527f50bd0fe09e5
Gerrit-Change-Number: 14899
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to