在 2021年11月16日星期二，Gao Xiang <<a href="mailto:hsiangkao@linux.alibaba.com">hsiangkao@linux.alibaba.com</a>> 写道：<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jianan,<br>
<br>
On Sat, Nov 13, 2021 at 12:09:33AM +0800, Huang Jianan wrote:<br>
> From: Huang Jianan <<a href="mailto:huangjianan@oppo.com">huangjianan@oppo.com</a>><br>
> <br>
> Renaming lz4_0padding to zero_padding globally since LZMA and later<br>
> algorithms also need that.<br>
> <br>
> Signed-off-by: Huang Jianan <<a href="mailto:huangjianan@oppo.com">huangjianan@oppo.com</a>><br>
<br>
I'm fine with renaming the original on-disk feature from lz4_0padding<br>
to zero_padding... but could we leave `support_0padding' as-is?<br>
<br>
My own concern is<br>
 1) it seems 'support_zero_padding' is not much better than<br>
    'support_0padding' but it causes somewhat longer lines...<br>
<br>
 2) it causes somewhat backporting overhead but with no real<br>
    benefits...<br>
<br>
Otherwise it looks good to me. If you agree on this, I could<br>
update this patch manually at the time when I apply this (maybe<br>
about this weekend).<br>
<br></blockquote><div>Hi Xiang,</div><div><br></div><div>I modified this because considering that it starts with a number, there may be problems with future expansion.<br></div><div><br></div><div>This may be overly worrying because we won’t use it as a  prefix of variable name.  So be free to use 'support_0padding'.</div><div><br></div><div>Thanks,</div><div>Jianan</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks,<br>
Gao Xiang<br>
<br>
</blockquote>