David Ribeiro Alves has posted comments on this change.

Change subject: Silence clang -Waddress-of-packed-member warning
......................................................................


Patch Set 5:

Doing this in just tablet means adding more cruft there (to do this for clang 
only). The warning seems broken (we're taking the address of something that is 
actually an int64_t typedef).

FWIW Chromium disabled this all over too.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I03518bcca43ea6c891a7f622972465c1ea87ce63
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <davidral...@gmail.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to