main: let's move ACTION_RUN test into initialize_runtime()
authorLennart Poettering <lennart@poettering.net>
Fri, 15 Dec 2017 16:52:33 +0000 (17:52 +0100)
committerLennart Poettering <lennart@poettering.net>
Fri, 15 Dec 2017 19:52:28 +0000 (20:52 +0100)
Let's hide this check inside the function and make it easier to follow
the general control flow of main().

src/core/main.c

index ccd35f1..4fd6f4e 100644 (file)
@@ -1917,6 +1917,9 @@ static int initialize_runtime(
          * - Some only apply when we first start up, but not when we reexecute
          */
 
+        if (arg_action != ACTION_RUN)
+                return 0;
+
         if (arg_system) {
                 /* Make sure we leave a core dump without panicing the kernel. */
                 install_crash_handler();
@@ -2479,14 +2482,12 @@ int main(int argc, char *argv[]) {
 
         log_execution_mode(&first_boot);
 
-        if (arg_action == ACTION_RUN) {
-                r = initialize_runtime(skip_setup,
-                                       &saved_rlimit_nofile,
-                                       &saved_rlimit_memlock,
-                                       &error_message);
-                if (r < 0)
-                        goto finish;
-        }
+        r = initialize_runtime(skip_setup,
+                               &saved_rlimit_nofile,
+                               &saved_rlimit_memlock,
+                               &error_message);
+        if (r < 0)
+                goto finish;
 
         r = manager_new(arg_system ? UNIT_FILE_SYSTEM : UNIT_FILE_USER,
                         arg_action == ACTION_TEST ? MANAGER_TEST_FULL : 0,