<HTML><BODY><div> </div><div> </div><div> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16088811591227109893_BODY"><div class="cl_900768"><div class="WordSection1_mr_css_attr"><p class="MsoNormal_mr_css_attr">Thanks for the fixes!</p><p class="MsoNormal_mr_css_attr"> </p><p class="MsoNormal_mr_css_attr">On 23.12.2020 14:14, mechanik20051988 wrote:</p><p class="MsoNormal_mr_css_attr">> 1. Fxed</p><p class="MsoNormal_mr_css_attr">> 2. As i understand you comment, you wrote that pools</p><p class="MsoNormal_mr_css_attr">> should be aligned on the 8-byte boundary.</p><p class="MsoNormal_mr_css_attr">> This is guaranteed by the granularity parameter in small_class if min_alloc</p><p class="MsoNormal_mr_css_attr">> have 8-byte boundary alignment. We calculate</p><p class="MsoNormal_mr_css_attr">> min_alloc = objsize_min = small_align(objsize_min, sizeof(intptr_t));</p><p class="MsoNormal_mr_css_attr">> and pass granularity sizeof(uintptr_t).</p><p class="MsoNormal_mr_css_attr"> </p><p class="MsoNormal_mr_css_attr">No, I meant the indentation alignment is broken. If a</p><p class="MsoNormal_mr_css_attr">function call consists of multiple lines, the non-first</p><p class="MsoNormal_mr_css_attr">lines should be aligned by '(' + 1. See examples in</p><p class="MsoNormal_mr_css_attr">the existing code.</p><div class="MsoNormal_mr_css_attr"> </div><div class="MsoNormal_mr_css_attr">Fixed:</div><div class="MsoNormal_mr_css_attr">+ small_class_create(&alloc->small_class, sizeof(intptr_t),<br>+ alloc->factor, objsize_min);<br>+ factor_pool_create(alloc);</div><div class="MsoNormal_mr_css_attr"> </div><p class="MsoNormal_mr_css_attr"> </p></div></div></div></div></div></div></div><div> </div></BODY></HTML>