dbwrap_tool: Simplify persistent/non-persistent check / dbwrap_tool: Avoid an unnecessary "else"

Andrew Bartlett abartlet at samba.org
Tue Sep 4 03:49:20 UTC 2018


On Tue, 2018-09-04 at 05:33 +0200, Volker Lendecke via samba-technical
wrote:
> On Tue, Sep 04, 2018 at 09:11:32AM +1200, Andrew Bartlett via samba-technical wrote:
> > 
> > What specifically was the goal of the change here?
> 
> Whenever I see an "else" with a "return" right before, the "else" is
> redundant. The "else" makes me go back to the "if-condition" and see
> what it was. Without the "else" this special condition is just done
> and out of my mind. Easier flow to read for me.

Thanks, that makes more sense now.

Andrew Bartlett

-- 
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team         https://samba.org
Samba Development and Support, Catalyst IT   
https://catalyst.net.nz/services/samba







More information about the samba-technical mailing list