<div dir="ltr"><a href="https://reviews.llvm.org/D60662">https://reviews.llvm.org/D60662</a> <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 14, 2019 at 4:02 PM Roman Lebedev <<a href="mailto:lebedev.ri@gmail.com">lebedev.ri@gmail.com</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">The commit message was<br>
> The last use of this helper method was removed by rL302385.<br>
Which is clearly not true, as this commit removed more uses of that method,<br>
that weren't testing the method itself, but using it.<br>
Therefore i'm questioning the whole removal.<br>
Moving it into a static function in tests is another, unrelated matter.<br>
<br>
Roman.<br>
<br>
On Sun, Apr 14, 2019 at 10:57 AM Fāng-ruì Sòng <<a href="mailto:maskray@google.com" target="_blank">maskray@google.com</a>> wrote:<br>
><br>
> On Sun, Apr 14, 2019 at 2:14 PM Roman Lebedev <<a href="mailto:lebedev.ri@gmail.com" target="_blank">lebedev.ri@gmail.com</a>> wrote:<br>
>><br>
>> Hello.<br>
>><br>
>> The function was clearly used in unit test, and the way<br>
>> the test was rewritten to avoid it looks more complicated<br>
>> than it was with the function, at least to me..<br>
>><br>
>> Was there any particular reason it was removed?<br>
>> If not, could you please remove last two commits?<br>
>><br>
>> Roman.<br>
><br>
><br>
> If the helper method simplifies the unittest, shall we define a static function in the unittest? It is not used (useful) in library code as isSizeStrictlySmallerThan/isSizeLargerThan are better.<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">宋方睿</div></div>