Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15795 )

Change subject: [warning] Fixed GCC warning "control reaches end of non-void 
function"
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15795/2/src/kudu/util/net/sockaddr.cc
File src/kudu/util/net/sockaddr.cc:

http://gerrit.cloudera.org:8080/#/c/15795/2/src/kudu/util/net/sockaddr.cc@154
PS2, Line 154:       // to quite GCC warning: control reaches end of non-void 
function
nit: I don't think we need the comment.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8c22cbc18892374966975bee9e50f4119580006b
Gerrit-Change-Number: 15795
Gerrit-PatchSet: 2
Gerrit-Owner: Volodymyr Verovkin <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Volodymyr Verovkin <[email protected]>
Gerrit-Comment-Date: Fri, 24 Apr 2020 15:16:36 +0000
Gerrit-HasComments: Yes

Reply via email to