player: return as soon as possible.

We need to take account of user side, not internal ones.

Change-Id: Ic7bccc343c771870d10b25fc7dec01217a1d4e35

diff --git a/src/lottie/lottieloader.cpp b/src/lottie/lottieloader.cpp
index 3d42195..b6323de 100644 (file)
--- a/src/lottie/lottieloader.cpp
+++ b/src/lottie/lottieloader.cpp
@@ -52,8 +52,6 @@ LottieLoader::LottieLoader()
 
 bool LottieLoader::load(std::string &path)
 {
-   if (path.empty()) return false;
-
    LottieFileCache &fileCache = LottieFileCache::get();
 
    mModel = fileCache.find(path);

diff --git a/src/lottie/lottieplayer.cpp b/src/lottie/lottieplayer.cpp
index b68e0f4..bda3213 100644 (file)
--- a/src/lottie/lottieplayer.cpp
+++ b/src/lottie/lottieplayer.cpp
@@ -94,6 +94,11 @@ LOTPlayerPrivate::LOTPlayerPrivate()
 bool
 LOTPlayerPrivate::setFilePath(std::string path)
 {
+   if (path.empty()) {
+      vWarning << "File path is empty";
+      return false;
+   }
+
    LottieLoader loader;
    if (loader.load(path)) {
       mModel = loader.model();