<div dir="ltr">Could you please commit this changes? <br><br>I have not found what i need to do next after recieve "approve" for changes in <a href="https://www.mesa3d.org/submittingpatches.html#reviewing">https://www.mesa3d.org/submittingpatches.html#reviewing</a>. <br>Do i need to send new mail with "<span style="font-size:12.8px">Reviewed-by: </span>" mark?</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 13, 2017 at 6:36 PM, Marek Olšák <span dir="ltr"><<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've changed my mind. The patch is OK:<br>
<br>
Reviewed-by: Marek Olšák <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>><br>
<span class="HOEnZb"><font color="#888888"><br>
Marek<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed, Sep 13, 2017 at 4:45 PM, Denis Pauk <<a href="mailto:pauk.denis@gmail.com">pauk.denis@gmail.com</a>> wrote:<br>
> What do you think about replace all checks in patch to asserts?<br>
><br>
><br>
> Best regards,<br>
>                   Denis.<br>
><br>
> On Sep 13, 2017 1:00 PM, "Marek Olšák" <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote:<br>
>><br>
>> On Wed, Sep 13, 2017 at 6:54 AM, Денис Паук <<a href="mailto:pauk.denis@gmail.com">pauk.denis@gmail.com</a>> wrote:<br>
>> > Do you mean delete check in u_format.c:: util_format_is_supported? Could<br>
>> > you<br>
>> > please explain more?<br>
>><br>
>> I mean delete everything. Invalid formats shouldn't be passed to<br>
>> pipe_screen::is_format_<wbr>supported. The code doing it needs to be fixed.<br>
>><br>
>> Marek<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Best regards,<br>                  Denis.</div>
</div>