<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi<div class=""><br class=""></div><div class="">I have a some code changes to fix warnings and potential portability issues related to <a class="gwt-InlineHyperlink" href="http://review.gluster.org/#/q/status:open+project:glusterfs+branch:master+topic:bug-1155181,n,z" style="color: rgb(68, 68, 68); text-decoration: none; font-family: 'Arial Unicode MS', Arial, sans-serif; font-size: small; background-color: rgb(255, 255, 255);">bug-1155181</a>, but would also show up on other platforms if compiling with (newer) clang.</div><div class=""><br class=""></div><div class="">Could I get you to review these </div><div class=""><a href="http://review.gluster.org/9520" class="">http://review.gluster.org/9520</a></div><div class=""><a href="http://review.gluster.org/9545" class="">http://review.gluster.org/9545</a></div><div class=""><a href="http://review.gluster.org/9549" class="">http://review.gluster.org/9549</a></div><div class=""><br class=""></div><div class="">Especially </div><div class=""><a href="http://review.gluster.org/#/c/9522/" class="">http://review.gluster.org/#/c/9522/</a></div><div class=""><br class=""></div><div class="">since it highlights some logic that isn’t quite.. logical.</div><div class=""><br class=""></div><div class="">cheers, </div><div class="">:-Dennis</div></body></html>