emotion: fix triggering condition of fallback module loading
authorWonguk Jeong <wonguk.jeong@samsung.com>
Sun, 20 Apr 2014 13:58:47 +0000 (09:58 -0400)
committerMike Blumenkrantz <zmike@samsung.com>
Sun, 20 Apr 2014 13:58:59 +0000 (09:58 -0400)
@fix

Summary:
In emotion_engine_instance_new(), if given module name is wrong or if loding is failed, fallback module loading should be triggered.a (previously, it check just whether name is NULL or not)

In case of T1025, "vlc" was given name. Therefore, fallback was not triggered.

Fixes T1025

Test Plan: enlightenment -> efm -> mouse over multimedia files -> check whether preview is shown properly or not

Reviewers: raster, cedric

CC: seoz, cedric, zmike
Maniphest Tasks: T1025

Differential Revision: https://phab.enlightenment.org/D773

src/lib/emotion/emotion_modules.c

index d10520df4460b7948b25397d58e61633533e741b..2e953870b4b6f480f98d0645371ca9cb625bf911 100644 (file)
@@ -356,16 +356,14 @@ emotion_engine_instance_new(const char *name, Evas_Object *obj, Emotion_Module_O
         m = _find_mod(name);
         if (m) eina_module_load(m);
      }
-   else
+
+   if (!_emotion_engine_registry)
      {
-        if (!_emotion_engine_registry)
-          {
-             m = _find_mod("generic");
-             if (!m) m = _find_mod("xine");
-             if (!m) m = _find_mod("gstreamer");
-             if (!m) m = _find_mod("gstreamer1");
-             if (m) eina_module_load(m);
-          }
+        m = _find_mod("generic");
+        if (!m) m = _find_mod("xine");
+        if (!m) m = _find_mod("gstreamer");
+        if (!m) m = _find_mod("gstreamer1");
+        if (m) eina_module_load(m);
      }
 
    if (name)