config: parse "primary_monitor_first"

Ignored, as tint2 has removed support for it already. It will at least print out a more specific error message. Related issues: - #35
jmc-88 · Sep 16, 2017 · 6b24a53 · 6b24a53
1 parent 2da459b
commit 6b24a53
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/config.cc b/src/config.cc
@@ -1714,6 +1714,10 @@ bool Reader::AddEntry_Legacy(std::string const& key, std::string const& value) {
     return true;
 #endif  // ENABLE_BATTERY
   }
+  if (key == "primary_monitor_first") {
+    util::log::Error() << "Ignoring legacy option \"primary_monitor_first\".\n";
+    return true;
+  }
 
   return false;
 }

diff --git a/src/config_test.cc b/src/config_test.cc
@@ -239,6 +239,9 @@ battery_font_color = #FFFFFF 74
 battery_padding = 1 0
 battery_background_id = 0
 
+# Ignored legacy options
+primary_monitor_first = 1
+
 # End of config)EOF";
 
 TEST_CASE("ConfigParser", "Correctly loads a valid configuration file") {