Application: Remove unnecessary command_line override

Author: ryonakano

src/Application.vala

@@ -23,15 +23,16 @@ namespace Cherrypick {
     public class Application : Gtk.Application {
         private Window? window;
         private Xdp.Portal portal;
+        private static bool is_immediately_pick = false;
 
         private OptionEntry[] CMD_OPTION_ENTRIES = {
-            {"immediately-pick", 'p', OptionFlags.NONE, OptionArg.NONE, null, _("Immediately pick a colour and copy it to clipboard"), null}
+            {"immediately-pick", 'p', OptionFlags.NONE, OptionArg.NONE, ref is_immediately_pick, _("Immediately pick a colour and copy it to clipboard"), null}
         };
 
         public Application () {
             Object (
                 application_id: "io.github.ellie_commons.cherrypick",
-                flags: ApplicationFlags.HANDLES_COMMAND_LINE
+                flags: ApplicationFlags.FLAGS_NONE
             );
         }
 
@@ -96,23 +97,13 @@ namespace Cherrypick {
             } else {
                     window.present ();
             }
-        }
-
-        public override int command_line (ApplicationCommandLine command) {
 
             /* Opens and immediately starts picking color if the --immediately-pick
                flag is passed when launching from the command line. This could
                be helpful for the user to set up keybindings and stuff */
-            var options = command.get_options_dict ();
-            if (options.contains ("immediately-pick")) {
+            if (Application.is_immediately_pick) {
                 immediately_pick ();
-
             }
-
-            //FIXME: Prevent window from showing if we do an immediate pick
-            //Quit or anything that makes Activate not do a window, end up in Picking failing
-            activate ();
-            return 0;
         }
 
         private void immediately_pick () {