minor fix

Author: shincurry

animeloop-cli/main.cpp

@@ -13,6 +13,7 @@
 
 #include "cxxopts.hpp"
 #include "loop_video.hpp"
+#include "utils.hpp"
 
 using namespace std;
 using namespace boost::filesystem;
@@ -55,13 +56,7 @@ int main(int argc, char * argv[]) {
             cout << endl;
             cout << ":: detecting external program..." << endl;
 
-            #ifdef _WIN32
-            string find_ffmpeg_command = "where /q ffmpeg";
-            #else
-            string find_ffmpeg_command = "which ffmpeg &> /dev/null";
-            #endif
-
-            if (system(find_ffmpeg_command.c_str()) == 0) {
+            if (detect_ffmpeg()) {
                 cout << "[o] detected ffmpeg." << endl;
             } else {
                 cout << "[x] not detect ffmpeg." << endl;

animeloop-cli/utils.cpp

@@ -52,7 +52,7 @@ void al::resize_video(path input_filepath, path output_filepath, Size size) {
 
     auto if_exists = exists(output_filepath);
     if (!if_exists) {
-        if (system("where /q ffmpeg") == 0) {
+        if (detect_ffmpeg()) {
             cout << ":: resizing video..." << endl;
             const string i = input_filepath.string();
             const string s = (to_string(size.width) + "x" + to_string(size.height));
@@ -308,4 +308,14 @@ std::string al::time_string(double seconds) {
     auto ms = boost::posix_time::milliseconds(seconds * 1000);
     auto time = boost::posix_time::time_duration(ms);
     return boost::posix_time::to_simple_string(time);
+}
+
+bool detect_ffmpeg() {
+    #ifdef _WIN32
+    string find_ffmpeg_command = "where /q ffmpeg";
+    #else
+    string find_ffmpeg_command = "which ffmpeg &> /dev/null";
+    #endif
+
+    return system(find_ffmpeg_command.c_str()) == 0;
 }

animeloop-cli/utils.hpp

@@ -57,4 +57,6 @@ namespace al {
     std::string time_string(double seconds);
 }
 
+bool detect_ffmpeg();
+
 #endif /* utils_hpp */