Merge pull request #221 from Libvisual/lv-tool-base-exclusion-set

lv-tool: Always skip actors "gstreamer" and "gdkpixbuf" when cycling

Author: hartwork

libvisual/tools/lv-tool/lv-tool.cpp

@@ -32,6 +32,7 @@
 #include <atomic>
 #include <stdexcept>
 #include <iostream>
+#include <unordered_set>
 #include <string>
 #include <cstdio>
 #include <cstdlib>
@@ -56,6 +57,11 @@
 
 namespace {
 
+  const std::unordered_set<std::string> actors_to_skip {
+      "gdkpixbuf",
+      "gstreamer"
+  };
+
   std::string actor_name;
   std::string input_name = DEFAULT_INPUT;
   std::string morph_name = DEFAULT_MORPH;
@@ -388,6 +394,12 @@ namespace {
           new_name = cycler (nullptr);
       }
 
+      // Always skip actors that are of little interest to end users,
+      // while allowing explicit "--actor (gdkpixbuf|gstreamer)".
+      if (actors_to_skip.contains (new_name)) {
+          return cycle_actor_name (new_name, dir);
+      }
+
       // FIXME: this won't work if an actor's name is used as part of
       // another actor's name
       if (exclude_actors.find (new_name) != std::string::npos) {