[PATCH] ctdb-server: Remove one-line goto

Andrew Bartlett abartlet at samba.org
Thu Mar 15 18:41:32 UTC 2018


On Thu, 2018-03-15 at 09:17 +0100, Swen Schillig via samba-technical
wrote:
> More thoughts ?
> 
> Cheers Swen.

My main thought is that reformatting code to README.Coding should be
done with care and great thought. 

There is much non-compliant code in Samba and while that isn't a great
situation we have not chosen so far to do a wholesale reformat. 

In particular changes like this consume abnormal amounts of review
resources and attract bike-shedding, dragging in even more folks to the
conversation. 

They are also more risky than you might expect, because this is the
single easiest way to introduce a regression, as it is 'only'
reformatting, but actually changes the control flow.

Your attention on Samba is most welcome, but I we are over-flowing with
end-user visible bugs, perhaps you could help with some of those, or
(at the more mechanical but valuable) the Python2->Python3 conversion?

Thanks,

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




More information about the samba-technical mailing list