options: fix 2 issues

1. Calling valueChanged didn't invoke machinery in
   value<t>, only base_value aka value_.

   There's a fast path in value<t>::type() despite the
   pessimization.

2. Split global scope stuff into options::globals from
   the options::globals stuff

3. Adjust usages

1 files changed, 3 insertions, 2 deletions

diff --git a/migration/20180102_00-process-detector-separator.cpp b/migration/20180102_00-process-detector-separator.cpp
index ddb01cb4..6c3a2b58 100644
--- a/migration/20180102_00-process-detector-separator.cpp
+++ b/migration/20180102_00-process-detector-separator.cpp
@@ -2,6 +2,7 @@
 #include "options/options.hpp"
 
 using namespace options;
+using namespace options::globals;
 using namespace migrations;
 
 static constexpr auto OLD_RECORD_SEPARATOR  = QChar('|');
@@ -26,7 +27,7 @@ struct process_detector_record_separator : migration
 
     bool should_run() const override
     {
-        return group::with_global_settings_object([](const QSettings& s)
+        return with_global_settings_object([](const QSettings& s)
         {
             const QString old_value = s.value(KEY_NAME).toString();
             return old_value.contains(OLD_RECORD_SEPARATOR);
@@ -35,7 +36,7 @@ struct process_detector_record_separator : migration
 
     void run() override
     {
-        return group::with_global_settings_object([](QSettings& s)
+        return with_global_settings_object([](QSettings& s)
         {
             QString value = s.value(KEY_NAME).toString();
             value.replace(OLD_UNIT_SEPARATOR,   NEW_UNIT_SEPARATOR);