mirror of
https://github.com/qemu/qemu.git
synced 2024-11-23 10:53:37 +08:00
qapi/ui: Drop temporary 'prefix'
Recent commit "qapi: Smarter camel_to_upper() to reduce need for 'prefix'" added a temporary 'prefix' to delay changing the generated code. Revert it. This improves DisplayGLMode's generated enumeration constant prefix from DISPLAYGL_MODE to DISPLAY_GL_MODE. Signed-off-by: Markus Armbruster <armbru@redhat.com> Message-ID: <20240904111836.3273842-9-armbru@redhat.com>
This commit is contained in:
parent
0aa4bd6aeb
commit
154fd4d1a0
@ -1397,7 +1397,6 @@
|
||||
# Since: 3.0
|
||||
##
|
||||
{ 'enum' : 'DisplayGLMode',
|
||||
'prefix' : 'DISPLAYGL_MODE', # TODO drop
|
||||
'data' : [ 'off', 'on', 'core', 'es' ] }
|
||||
|
||||
##
|
||||
|
@ -1971,7 +1971,7 @@ static void qemu_create_early_backends(void)
|
||||
|
||||
qemu_console_early_init();
|
||||
|
||||
if (dpy.has_gl && dpy.gl != DISPLAYGL_MODE_OFF && display_opengl == 0) {
|
||||
if (dpy.has_gl && dpy.gl != DISPLAY_GL_MODE_OFF && display_opengl == 0) {
|
||||
#if defined(CONFIG_OPENGL)
|
||||
error_report("OpenGL is not supported by display backend '%s'",
|
||||
DisplayType_str(dpy.type));
|
||||
|
@ -176,7 +176,7 @@ dbus_display_add_console(DBusDisplay *dd, int idx, Error **errp)
|
||||
assert(con);
|
||||
|
||||
if (qemu_console_is_graphic(con) &&
|
||||
dd->gl_mode != DISPLAYGL_MODE_OFF) {
|
||||
dd->gl_mode != DISPLAY_GL_MODE_OFF) {
|
||||
qemu_console_set_display_gl_ctx(con, &dd->glctx);
|
||||
}
|
||||
|
||||
@ -466,9 +466,9 @@ static const TypeInfo dbus_vc_type_info = {
|
||||
static void
|
||||
early_dbus_init(DisplayOptions *opts)
|
||||
{
|
||||
DisplayGLMode mode = opts->has_gl ? opts->gl : DISPLAYGL_MODE_OFF;
|
||||
DisplayGLMode mode = opts->has_gl ? opts->gl : DISPLAY_GL_MODE_OFF;
|
||||
|
||||
if (mode != DISPLAYGL_MODE_OFF) {
|
||||
if (mode != DISPLAY_GL_MODE_OFF) {
|
||||
#ifdef CONFIG_OPENGL
|
||||
egl_init(opts->u.dbus.rendernode, mode, &error_fatal);
|
||||
#else
|
||||
@ -482,7 +482,7 @@ early_dbus_init(DisplayOptions *opts)
|
||||
static void
|
||||
dbus_init(DisplayState *ds, DisplayOptions *opts)
|
||||
{
|
||||
DisplayGLMode mode = opts->has_gl ? opts->gl : DISPLAYGL_MODE_OFF;
|
||||
DisplayGLMode mode = opts->has_gl ? opts->gl : DISPLAY_GL_MODE_OFF;
|
||||
|
||||
if (opts->u.dbus.addr && opts->u.dbus.p2p) {
|
||||
error_report("dbus: can't accept both addr=X and p2p=yes options");
|
||||
|
@ -17,7 +17,7 @@ QEMUGLContext qemu_egl_create_context(DisplayGLCtx *dgc,
|
||||
EGL_CONTEXT_MINOR_VERSION_KHR, params->minor_ver,
|
||||
EGL_NONE
|
||||
};
|
||||
bool gles = (qemu_egl_mode == DISPLAYGL_MODE_ES);
|
||||
bool gles = (qemu_egl_mode == DISPLAY_GL_MODE_ES);
|
||||
|
||||
ctx = eglCreateContext(qemu_egl_display, qemu_egl_config,
|
||||
eglGetCurrentContext(),
|
||||
|
@ -207,7 +207,7 @@ static const DisplayGLCtxOps eglctx_ops = {
|
||||
|
||||
static void early_egl_headless_init(DisplayOptions *opts)
|
||||
{
|
||||
DisplayGLMode mode = DISPLAYGL_MODE_ON;
|
||||
DisplayGLMode mode = DISPLAY_GL_MODE_ON;
|
||||
|
||||
if (opts->has_gl) {
|
||||
mode = opts->gl;
|
||||
|
@ -503,7 +503,7 @@ static int qemu_egl_init_dpy(EGLNativeDisplayType dpy,
|
||||
EGLint major, minor;
|
||||
EGLBoolean b;
|
||||
EGLint n;
|
||||
bool gles = (mode == DISPLAYGL_MODE_ES);
|
||||
bool gles = (mode == DISPLAY_GL_MODE_ES);
|
||||
|
||||
qemu_egl_display = qemu_egl_get_display(dpy, platform);
|
||||
if (qemu_egl_display == EGL_NO_DISPLAY) {
|
||||
@ -533,7 +533,7 @@ static int qemu_egl_init_dpy(EGLNativeDisplayType dpy,
|
||||
return -1;
|
||||
}
|
||||
|
||||
qemu_egl_mode = gles ? DISPLAYGL_MODE_ES : DISPLAYGL_MODE_CORE;
|
||||
qemu_egl_mode = gles ? DISPLAY_GL_MODE_ES : DISPLAY_GL_MODE_CORE;
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -564,8 +564,8 @@ int qemu_egl_init_dpy_mesa(EGLNativeDisplayType dpy, DisplayGLMode mode)
|
||||
int qemu_egl_init_dpy_win32(EGLNativeDisplayType dpy, DisplayGLMode mode)
|
||||
{
|
||||
/* prefer GL ES, as that's what ANGLE supports */
|
||||
if (mode == DISPLAYGL_MODE_ON) {
|
||||
mode = DISPLAYGL_MODE_ES;
|
||||
if (mode == DISPLAY_GL_MODE_ON) {
|
||||
mode = DISPLAY_GL_MODE_ES;
|
||||
}
|
||||
|
||||
if (qemu_egl_init_dpy(dpy, 0, mode) < 0) {
|
||||
@ -618,7 +618,7 @@ EGLContext qemu_egl_init_ctx(void)
|
||||
EGL_CONTEXT_CLIENT_VERSION, 2,
|
||||
EGL_NONE
|
||||
};
|
||||
bool gles = (qemu_egl_mode == DISPLAYGL_MODE_ES);
|
||||
bool gles = (qemu_egl_mode == DISPLAY_GL_MODE_ES);
|
||||
EGLContext ectx;
|
||||
EGLBoolean b;
|
||||
|
||||
@ -642,7 +642,7 @@ bool egl_init(const char *rendernode, DisplayGLMode mode, Error **errp)
|
||||
{
|
||||
ERRP_GUARD();
|
||||
|
||||
if (mode == DISPLAYGL_MODE_OFF) {
|
||||
if (mode == DISPLAY_GL_MODE_OFF) {
|
||||
error_setg(errp, "egl: turning off GL doesn't make sense");
|
||||
return false;
|
||||
}
|
||||
|
4
ui/gtk.c
4
ui/gtk.c
@ -2514,7 +2514,7 @@ static void early_gtk_display_init(DisplayOptions *opts)
|
||||
}
|
||||
|
||||
assert(opts->type == DISPLAY_TYPE_GTK);
|
||||
if (opts->has_gl && opts->gl != DISPLAYGL_MODE_OFF) {
|
||||
if (opts->has_gl && opts->gl != DISPLAY_GL_MODE_OFF) {
|
||||
#if defined(CONFIG_OPENGL)
|
||||
#if defined(GDK_WINDOWING_WAYLAND)
|
||||
if (GDK_IS_WAYLAND_DISPLAY(gdk_display_get_default())) {
|
||||
@ -2530,7 +2530,7 @@ static void early_gtk_display_init(DisplayOptions *opts)
|
||||
#endif
|
||||
{
|
||||
#ifdef CONFIG_X11
|
||||
DisplayGLMode mode = opts->has_gl ? opts->gl : DISPLAYGL_MODE_ON;
|
||||
DisplayGLMode mode = opts->has_gl ? opts->gl : DISPLAY_GL_MODE_ON;
|
||||
gtk_egl_init(mode);
|
||||
#endif
|
||||
}
|
||||
|
@ -147,11 +147,11 @@ QEMUGLContext sdl2_gl_create_context(DisplayGLCtx *dgc,
|
||||
SDL_GL_MakeCurrent(scon->real_window, scon->winctx);
|
||||
|
||||
SDL_GL_SetAttribute(SDL_GL_SHARE_WITH_CURRENT_CONTEXT, 1);
|
||||
if (scon->opts->gl == DISPLAYGL_MODE_ON ||
|
||||
scon->opts->gl == DISPLAYGL_MODE_CORE) {
|
||||
if (scon->opts->gl == DISPLAY_GL_MODE_ON ||
|
||||
scon->opts->gl == DISPLAY_GL_MODE_CORE) {
|
||||
SDL_GL_SetAttribute(SDL_GL_CONTEXT_PROFILE_MASK,
|
||||
SDL_GL_CONTEXT_PROFILE_CORE);
|
||||
} else if (scon->opts->gl == DISPLAYGL_MODE_ES) {
|
||||
} else if (scon->opts->gl == DISPLAY_GL_MODE_ES) {
|
||||
SDL_GL_SetAttribute(SDL_GL_CONTEXT_PROFILE_MASK,
|
||||
SDL_GL_CONTEXT_PROFILE_ES);
|
||||
}
|
||||
@ -163,7 +163,7 @@ QEMUGLContext sdl2_gl_create_context(DisplayGLCtx *dgc,
|
||||
/* If SDL fail to create a GL context and we use the "on" flag,
|
||||
* then try to fallback to GLES.
|
||||
*/
|
||||
if (!ctx && scon->opts->gl == DISPLAYGL_MODE_ON) {
|
||||
if (!ctx && scon->opts->gl == DISPLAY_GL_MODE_ON) {
|
||||
SDL_GL_SetAttribute(SDL_GL_CONTEXT_PROFILE_MASK,
|
||||
SDL_GL_CONTEXT_PROFILE_ES);
|
||||
ctx = SDL_GL_CreateContext(scon->real_window);
|
||||
|
@ -107,7 +107,7 @@ void sdl2_window_create(struct sdl2_console *scon)
|
||||
if (scon->opengl) {
|
||||
const char *driver = "opengl";
|
||||
|
||||
if (scon->opts->gl == DISPLAYGL_MODE_ES) {
|
||||
if (scon->opts->gl == DISPLAY_GL_MODE_ES) {
|
||||
driver = "opengles2";
|
||||
}
|
||||
|
||||
|
@ -840,7 +840,7 @@ static void qemu_spice_init(void)
|
||||
"incompatible with -spice port/tls-port");
|
||||
exit(1);
|
||||
}
|
||||
egl_init(qemu_opt_get(opts, "rendernode"), DISPLAYGL_MODE_ON, &error_fatal);
|
||||
egl_init(qemu_opt_get(opts, "rendernode"), DISPLAY_GL_MODE_ON, &error_fatal);
|
||||
spice_opengl = 1;
|
||||
}
|
||||
#endif
|
||||
|
Loading…
Reference in New Issue
Block a user