<div dir="ltr"><div>How do you feel about the mix usage of tab and leading white spaces?</div><div>The code is mangled because some liens use a tab and some use white spaces.</div><div>I do not think the choice of tab vs white spaces is code representation. <br></div><div><br><div dir="ltr"><font size="1"><table style="display:inline"><tbody style="display:inline"><tr style="display:inline"><td style="font-family:Roboto,RobotoDraft,Helvetica,Arial,sans-serif;margin:0px;width:288px;vertical-align:top;display:inline"><table style="display:inline"><tbody style="display:inline"><tr style="display:inline"><td style="font-family:Roboto,RobotoDraft,Helvetica,Arial,sans-serif;margin:0px;width:288px;vertical-align:top;display:inline"><br></td></tr></tbody></table></td></tr></tbody></table></font></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 11, 2019 at 2:05 PM Simon Kelley <<a href="mailto:simon@thekelleys.org.uk">simon@thekelleys.org.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">There seems to be confusion here between code style, and code<br>
representation.<br>
<br>
Code style, ie layout, indent width, is not negotiable.<br>
<br>
Code representation is the use of tab characters at 8-character stops.<br>
If your IDE or other device doesn't use 8 characher stops, it will<br>
mangle code. That doesn't mean the code is badly formatted, it means<br>
your display method is incompatible.<br>
<br>
It's quite possible to get around this by running expand on every<br>
commit, so that only spaces are used in the repo. It's even possible to<br>
run expand on every existing file, at the expense of polluting the logs.<br>
I don't think there are any other downsides to this.<br>
<br>
Or people could  configure IDEs and editors to use sane tab stops.<br>
<br>
Cheers,<br>
<br>
Simon.<br>
<br>
On 07/09/2019 18:21, john doe wrote:<br>
> On 9/7/2019 6:25 PM, Shota Hino wrote:<br>
>> Whatever the width of the tab is, converting all tabs to whitespaces (or<br>
>> the other way around) would be better.<br>
>> If code formatting was forced at the time of each commit, there would be no<br>
>> need for anybody to set the tab width on their editor.<br>
>> Consistent coding style will help more developers in the future.<br>
>><br>
>><br>
>><br>
> <br>
> I agree, consistent code is best, clear guideline could be usefull for<br>
> new code.<br>
> Simon Kelley, the belligerent dictator of the Dnsmasq project will need<br>
> to way in on such changes though.<br>
> <br>
> --<br>
> John Doe<br>
> <br>
> _______________________________________________<br>
> Dnsmasq-discuss mailing list<br>
> <a href="mailto:Dnsmasq-discuss@lists.thekelleys.org.uk" target="_blank">Dnsmasq-discuss@lists.thekelleys.org.uk</a><br>
> <a href="http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss" rel="noreferrer" target="_blank">http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss</a><br>
> <br>
<br>
_______________________________________________<br>
Dnsmasq-discuss mailing list<br>
<a href="mailto:Dnsmasq-discuss@lists.thekelleys.org.uk" target="_blank">Dnsmasq-discuss@lists.thekelleys.org.uk</a><br>
<a href="http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss" rel="noreferrer" target="_blank">http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss</a><br>
</blockquote></div>