Hi Francesco,
Where are you with the piglit tests? I just finished converting the
ARB_viewport_array tests, and was thinking of having a go at the
ARB_texture_view ones. However if you've made significant progress
there already, I have other things I can do too.
-ilia
On Wed, Aug 31, 2016 at 1:43 AM, Francesco Ansanelli
<franci...@gmail.com> wrote:
> Hi,
>
> I sent this series to have some feedback (rfc prefix failed) and the comment
> will be addressed as soon as the tests are done.
> I started checking the piglit part that you suggested in a previous mail..
>
> I'll ask you also about them if I don't bother too much :)
>
>
> Il 31 ago 2016 01:44, "Ilia Mirkin" <imir...@alum.mit.edu> ha scritto:
>>
>> On Mon, Aug 29, 2016 at 1:25 AM, Francesco Ansanelli
>> <franci...@gmail.com> wrote:
>> > XXX still need to figure how to treat the removed VIEW_CLASS*
>> > and formats.
>>
>> Can you elaborate what this comment means?
>>
>> You definitely need to add piglit tests for testing ETC2 stuff - it's
>> not supported in hardware for most desktop hw, and so a fallback
>> method is used. I think that's the main thing this patchset is waiting
>> on...
>>
>> -ilia
>>
>> >
>> > V2: drop the oes suffix in messages
>> > (Ilia Mirkin)
>> >
>> > Signed-off-by: Francesco Ansanelli <franci...@gmail.com>
>> > ---
>> > src/mesa/main/textureview.c | 11 +++++++----
>> > 1 file changed, 7 insertions(+), 4 deletions(-)
>> >
>> > diff --git a/src/mesa/main/textureview.c b/src/mesa/main/textureview.c
>> > index ed66c17..36663cc 100644
>> > --- a/src/mesa/main/textureview.c
>> > +++ b/src/mesa/main/textureview.c
>> > @@ -387,8 +387,10 @@ target_valid(struct gl_context *ctx, GLenum
>> > origTarget, GLenum newTarget)
>> > switch (origTarget) {
>> > case GL_TEXTURE_1D:
>> > case GL_TEXTURE_1D_ARRAY:
>> > - RETURN_IF_SUPPORTED(TEXTURE_1D);
>> > - RETURN_IF_SUPPORTED(TEXTURE_1D_ARRAY);
>> > + if (!_mesa_is_gles3(ctx)) {
>> > + RETURN_IF_SUPPORTED(TEXTURE_1D);
>> > + RETURN_IF_SUPPORTED(TEXTURE_1D_ARRAY);
>> > + }
>> > break;
>> > case GL_TEXTURE_2D:
>> > RETURN_IF_SUPPORTED(TEXTURE_2D);
>> > @@ -398,7 +400,8 @@ target_valid(struct gl_context *ctx, GLenum
>> > origTarget, GLenum newTarget)
>> > RETURN_IF_SUPPORTED(TEXTURE_3D);
>> > break;
>> > case GL_TEXTURE_RECTANGLE:
>> > - RETURN_IF_SUPPORTED(TEXTURE_RECTANGLE);
>> > + if (!_mesa_is_gles3(ctx))
>> > + RETURN_IF_SUPPORTED(TEXTURE_RECTANGLE);
>> > break;
>> > case GL_TEXTURE_CUBE_MAP:
>> > case GL_TEXTURE_2D_ARRAY:
>> > @@ -514,7 +517,7 @@ _mesa_set_texture_view_state(struct gl_context *ctx,
>> > }
>> >
>> > /**
>> > - * glTextureView (ARB_texture_view)
>> > + * glTextureView (ARB_texture_view / OES_texture_view)
>> > * If an error is found, record it with _mesa_error()
>> > * \return none.
>> > */
>> > --
>> > 1.7.9.5
>> >
>> > _______________________________________________
>> > mesa-dev mailing list
>> > mesa-dev@lists.freedesktop.org
>> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
