kudu-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexey Serbin (Code Review)" <ger...@cloudera.org>
Subject [kudu-CR] small cleanup: remove unnecessary semicolons (part 3)
Date Thu, 05 Dec 2019 22:00:37 GMT
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14847
)

Change subject: small cleanup: remove unnecessary semicolons (part 3)
......................................................................

small cleanup: remove unnecessary semicolons (part 3)

The motivation for this change was seeing a few warnings generated by
the built-in static analysis tools while browsing the code in QtCreator.

This patch does not contain any functional changes.

Change-Id: Id5b0e9f9f34030e4ede2dc5d2d256d386086b716
Reviewed-on: http://gerrit.cloudera.org:8080/14847
Reviewed-by: Adar Dembo <adar@cloudera.com>
Tested-by: Kudu Jenkins
---
M src/kudu/util/env_posix.cc
1 file changed, 4 insertions(+), 3 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified

-- 
To view, visit http://gerrit.cloudera.org:8080/14847
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Id5b0e9f9f34030e4ede2dc5d2d256d386086b716
Gerrit-Change-Number: 14847
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <aserbin@cloudera.com>
Gerrit-Reviewer: Adar Dembo <adar@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aserbin@cloudera.com>
Gerrit-Reviewer: Bankim Bhavsar <bankim@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

Mime
  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message