Skip to content
This repository was archived by the owner on Jun 16, 2022. It is now read-only.

tests/gl_basic: Fix compilation for surfaceless_egl#47

Open
abrodkin wants to merge 1 commit into
waffle-gl:masterfrom
abrodkin:topic-surfaceless_egl_compilation
Open

tests/gl_basic: Fix compilation for surfaceless_egl#47
abrodkin wants to merge 1 commit into
waffle-gl:masterfrom
abrodkin:topic-surfaceless_egl_compilation

Conversation

@abrodkin

Copy link
Copy Markdown
Contributor

If the only platform selected is surfaceless_egl test_gl_basic_glesX_fwdcompat
is not defined which make compiler unhappy:
--------------------------------------->8-------------------------------------
.../tests/functional/gl_basic_test.c:581:24: error: 'test_gl_basic_gles2_fwdcompat' undeclared (first use in this function); did you mean 'test_gl_basic_gles1_fwdcompat'?
unit_test_make(test_gl_basic_gles2_fwdcompat),
^
--------------------------------------->8-------------------------------------

Not really sure to which extent fwdcompact is supported in
surfaceless_egl but an educated guess is it's pretty much the same as
for GBM.

Also while at it fixing wrong comment.

Signed-off-by: Alexey Brodkin abrodkin@synopsys.com
Cc: Gurchetan Singh gurchetansingh@chromium.org
Cc: Emil Velikov emil.l.velikov@gmail.com
Cc: Chad Versace chadversary@chromium.org
Cc: Erico Nunes nunes.erico@gmail.com
Cc: Haixia Shi hshi@chromium.org

If the only platform selected is surfaceless_egl test_gl_basic_glesX_fwdcompat
is not defined which make compiler unhappy:
--------------------------------------->8-------------------------------------
.../tests/functional/gl_basic_test.c:581:24: error: 'test_gl_basic_gles2_fwdcompat' undeclared (first use in this function); did you mean 'test_gl_basic_gles1_fwdcompat'?
         unit_test_make(test_gl_basic_gles2_fwdcompat),                  \
                        ^
--------------------------------------->8-------------------------------------

Not really sure to which extent fwdcompact is supported in
surfaceless_egl but an educated guess is it's pretty much the same as
for GBM.

Also while at it fixing wrong comment.

Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
Cc: Gurchetan Singh <gurchetansingh@chromium.org>
Cc: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Chad Versace <chadversary@chromium.org>
Cc: Erico Nunes <nunes.erico@gmail.com>
Cc: Haixia Shi <hshi@chromium.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant