From: Sanghyup Lee <sh53.lee@samsung.com>
Date: Thu, 25 Jul 2013 02:03:01 +0000 (+0900)
Subject: Remove unnecessary repetition of initializing feedback library.
X-Git-Tag: 2.2_release~26
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dcc0bdb43b9794b9935418e7d7b6c901e1de73a4;p=framework%2Fweb%2Fwebkit-efl.git

Remove unnecessary repetition of initializing feedback library.

[Title] Remove unnecessary repetition of initializing feedback library.
[Issue#] N/A
[Problem] N/A
[Cause] We don't need to initialize feedback library each time.
[Solution] Move initialize method into getHandle method that called only one time. and remove deinitialize.

Change-Id: I07152592eed156f5c5ffcc648802fe7946a7004d
---

diff --git a/Source/WebCore/platform/efl/tizen/TizenLinkEffect.cpp b/Source/WebCore/platform/efl/tizen/TizenLinkEffect.cpp
index 6158a09..87faff3 100644
--- a/Source/WebCore/platform/efl/tizen/TizenLinkEffect.cpp
+++ b/Source/WebCore/platform/efl/tizen/TizenLinkEffect.cpp
@@ -32,9 +32,7 @@
 
 namespace WebCore {
 
-static int (*feedback_initialize)();
 static int (*feedback_play)(int);
-static int (*feedback_deinitialize)();
 
 static void* getHandle()
 {
@@ -42,7 +40,7 @@ static void* getHandle()
     if (!handle)
         return 0;
 
-    feedback_initialize = (int(*)(void))dlsym(handle, "feedback_initialize");
+    int (*feedback_initialize)() = (int(*)(void))dlsym(handle, "feedback_initialize");
     if (!feedback_initialize) {
         dlclose(handle);
         return 0;
@@ -54,12 +52,7 @@ static void* getHandle()
         return 0;
     }
 
-    feedback_deinitialize = (int(*)(void))dlsym(handle, "feedback_deinitialize");
-    if (!feedback_deinitialize) {
-        dlclose(handle);
-        return 0;
-    }
-
+    feedback_initialize();
     return handle;
 }
 
@@ -70,9 +63,7 @@ void TizenLinkEffect::playLinkEffect()
     if (!handle)
         return;
 
-    feedback_initialize();
     feedback_play(FEEDBACK_PATTERN_TAP);
-    feedback_deinitialize();
 }
 
 }