mi: Hush extension initialization (#75870)
authorAdam Jackson <ajax@redhat.com>
Thu, 27 Mar 2014 15:48:39 +0000 (11:48 -0400)
committerKeith Packard <keithp@keithp.com>
Sat, 29 Mar 2014 07:01:11 +0000 (00:01 -0700)
Printing these as ErrorF is fairly obnoxious, since it means the
non-hardware servers now spew stuff to the console for entirely routine
events.  And actually, printing these at all is fairly obnoxious, since
a) we're printing a line for every extension, whether it's enabled or
not, and b) we're not actually initializing the extension at this point.

Signed-off-by: Adam Jackson <ajax@redhat.com>
Reviewed-by: Daniel Stone <daniel@fooishbar.org>
Signed-off-by: Keith Packard <keithp@keithp.com>
mi/miinitext.c

index 5b45ab4..1d90516 100644 (file)
@@ -365,7 +365,6 @@ void
 LoadExtensionList(const ExtensionModule ext[], int size, Bool builtin)
 {
     ExtensionModule *newext;
-    const char *msg;
     int i;
 
     /* Make sure built-in extensions get added to the list before those
@@ -375,14 +374,7 @@ LoadExtensionList(const ExtensionModule ext[], int size, Bool builtin)
     if (!(newext = NewExtensionModuleList(size)))
         return;
 
-    if (builtin)
-        msg = "Initializing built-in";
-    else
-        msg = "Loading";
-
     for (i = 0; i < size; i++, newext++) {
-        ErrorF("%s extension %s\n", msg, ext[i].name);
-
         newext->name = ext[i].name;
         newext->initFunc = ext[i].initFunc;
         newext->disablePtr = ext[i].disablePtr;