<div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hello,<br></div><div><br></div><div>Thanks for review.</div><div>Could somebody help me with a push?</div><div><br></div><div>@Emil:<br></div><div>Yes, you are right about missing 'in-' in 'git send-email --in-reply-to='. <br></div><div>So v3 wasn't added to the patchwork(</div><div>Hope it isn't really mandatory and the push can be performed)<br></div><div><br></div><div>Thanks,</div><div>Andrii.<br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 11, 2019 at 3:14 PM Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@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">On 2018/12/12, <a href="mailto:asimiklit.work@gmail.com" target="_blank">asimiklit.work@gmail.com</a> wrote:<br>
> From: Andrii Simiklit <<a href="mailto:andrii.simiklit@globallogic.com" target="_blank">andrii.simiklit@globallogic.com</a>><br>
> <br>
> Test that usage of upside down miptree doesn't cause assertion<br>
> <br>
> The miptree:<br>
> <br>
> level 0 = 1x1<br>
> level 1 = 2x2<br>
> level 2 = 4x4<br>
> ...<br>
> level n = NxN<br>
> <br>
> should be acceptable for case when we don't use a min filter.<br>
> <br>
> v2: - Unnecessary function calls were removed<br>
>     - The 'glClearColor' call was moved to 'piglit_display' function<br>
>     - The program creation was moved to 'piglit_init' function<br>
>     - The requirements check was moved to 'piglit_init' function<br>
>                ( Erik Faye-Lund <<a href="mailto:erik.faye-lund@collabora.com" target="_blank">erik.faye-lund@collabora.com</a>> )<br>
> <br>
>     - Fixed a leak of texture which is created in 'setup_texture'<br>
> <br>
> v3: - Defined a constant for the test color instead of a hardcoded variant<br>
>     - Removed unused variable 'pos' which is always was 0<br>
>     - Implemented more effective number of levels calculator<br>
>     - Removed a stop when one of tests fails<br>
>     - Fixed two style issues<br>
>                ( Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>> )<br>
Looks great, thank you.<br>
<br>
Reviewed-by: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>><br>
<br>
-Emil<br>
<br>
P.S. Seems like you're missing "in-" from the git send-email --in-reply-to="old-thread-id" ...<br>
<br>
</blockquote></div>