<div dir="ltr">Hello everyone,
<br><br>I&#39;d like to bring some points up for discussion:

<br>1 - We have a great number of methods all across Gluster which (for 
example) are declared to be &#39;int&#39; but in reality return (for example) 0 
in all cases, both on the positive and the negative path.
<br>2 - In many cases there are calls to methods that return a value, but 
this value is completely ignored (e.g. the method &#39;foo(...)&#39; return an 
int, but all calls are in the form of &#39;foo(...)&#39;  instead of ret = 
&#39;foo(...)&#39; ).

<br><br>These 2 issues adversely affects the readability/maintainability of the 
project and probably also affect error handling (if an error occurs in a
 method that always returns 0, how will the calling methods know that an
 error happened?).
<br>These issues are not specific to a certain component, but seem to be all
 across the project.

<br><br>I believe this needs addressing, and probably in one of the following ways:
<br>1 - Change methods to &#39;void&#39;, where there is really no need for a return value.<br><div>2 - Actually handle the return value and log erros accordingly. <br></div><div><br></div><div>Of course discretion is needed and in some cases we could change to 
void, while in other handle the return value. <br></div><div>I&#39;d like to hear opinions on if and how this needs to be handled. <br></div><div><br></div><div>Thank you all.</div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div style="margin-left:44px"><div style="margin-bottom:0px;margin-left:0px;padding-bottom:5px"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><font face="RedHatText, sans-serif"><span style="font-size:14px;text-transform:capitalize"><b>Barak Sason Rofman</b></span></font><br></div></div></div></div></div></div></div></div></div></div><div style="margin-left:44px"><div style="margin-bottom:0px;margin-left:0px;padding-bottom:5px"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><p style="margin:0px;font-size:14px;color:rgb(0,0,0);font-family:overpass,sans-serif;font-weight:bold;padding:0px;text-transform:uppercase"><span style="font-family:RedHatText,sans-serif;font-size:12px;text-transform:capitalize;font-weight:normal;color:rgb(34,34,34)">Gluster Storage </span><span style="font-family:RedHatText,sans-serif;font-size:12px;text-transform:capitalize;font-weight:normal;color:rgb(34,34,34)">Development</span></p></div></div></div></div></div></div></div></div></div></div><div style="margin-left:44px"><div style="margin-bottom:0px;margin-left:0px;padding-bottom:5px"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><p style="margin:0px 0px 4px;font-size:12px;font-family:RedHatText,sans-serif"><a href="https://www.redhat.com/" style="color:rgb(0,136,206);margin:0px" target="_blank">Red Hat Israel</a></p><div style="margin-bottom:4px"><p style="margin:0px;font-size:12px;font-family:RedHatText,sans-serif">34 Jerusalem rd. Ra&#39;anana, 43501</p></div><p style="margin:0px;font-size:12px;font-family:RedHatText,sans-serif"><span style="margin:0px;padding:0px">bsasonro<a href="mailto:adi@redhat.com" style="color:rgb(0,0,0);margin:0px" target="_blank">@redhat.com</a>   </span> T: <u>+972-9-7692304</u><br>M: <u>+972-52-4326355</u></p><div style="margin-top:12px"><table border="0"><tbody><tr><td width="100px"><a href="https://red.ht/sig" style="color:rgb(17,85,204)" target="_blank"><img src="https://static.redhat.com/libs/redhat/brand-assets/latest/corp/logo.png" width="90" height="auto"></a></td></tr></tbody></table></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>