diff options
| author | Olivier Fourdan <ofourdan@redhat.com> | 2020-09-08 18:34:48 +0200 |
|---|---|---|
| committer | Olivier Fourdan <fourdan@gmail.com> | 2021-01-29 12:52:09 +0000 |
| commit | a81c98e42a954580ca0d43c89169dc2069c87b96 (patch) | |
| tree | 72cc6756100d5870ed2a75a323bcb0abbb8fb255 | |
| parent | 4341f1da728e4c67187b48865faf06c1312bcdff (diff) | |
mi: Don't log the full extension list on disable
When enabling or disabling an extension which is not known to the
Xserver, it will log an error message and list the extensions it knows
about.
That clutters the logs when the Xserver is Xwayland spawned by the
Wayland compositor who doesn't actually know the list of extensions
enabled at build time in the Xserver.
Considering that disabling a non-existing extension is a no-op anyway,
list all the extensions available only when attempting to enable an
extension which the Xserver doesn't know about.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
Reviewed-by: Michel Dänzer <mdaenzer@redhat.com>
| -rw-r--r-- | mi/miinitext.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/mi/miinitext.c b/mi/miinitext.c index b7c702127..358d485b3 100644 --- a/mi/miinitext.c +++ b/mi/miinitext.c @@ -221,8 +221,12 @@ EnableDisableExtensionError(const char *name, Bool enable) break; } } - if (found == FALSE) + if (found == FALSE) { ErrorF("[mi] Extension \"%s\" is not recognized\n", name); + /* Disabling a non-existing extension is a no-op anyway */ + if (enable == FALSE) + return; + } ErrorF("[mi] Only the following extensions can be run-time %s:\n", enable ? "enabled" : "disabled"); for (i = 0; i < ARRAY_SIZE(staticExtensions); i++) { |