On 2018/12/12, asimiklit.w...@gmail.com wrote:
> From: Andrii Simiklit <andrii.simik...@globallogic.com>
> 
> Test that usage of upside down miptree doesn't cause assertion
> 
> The miptree:
> 
> level 0 = 1x1
> level 1 = 2x2
> level 2 = 4x4
> ...
> level n = NxN
> 
> should be acceptable for case when we don't use a min filter.
> 
> v2: - Unnecessary function calls were removed
>     - The 'glClearColor' call was moved to 'piglit_display' function
>     - The program creation was moved to 'piglit_init' function
>     - The requirements check was moved to 'piglit_init' function
>                ( Erik Faye-Lund <erik.faye-l...@collabora.com> )
> 
>     - Fixed a leak of texture which is created in 'setup_texture'
> 
> v3: - Defined a constant for the test color instead of a hardcoded variant
>     - Removed unused variable 'pos' which is always was 0
>     - Implemented more effective number of levels calculator
>     - Removed a stop when one of tests fails
>     - Fixed two style issues
>                ( Emil Velikov <emil.l.veli...@gmail.com> )
Looks great, thank you.

Reviewed-by: Emil Velikov <emil.l.veli...@gmail.com>

-Emil

P.S. Seems like you're missing "in-" from the git send-email 
--in-reply-to="old-thread-id" ...

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to