lib: simplify 'umask'

Just check: if 'mask' is not undefined, just call 'validateMode' and
then return the unmask value, we don't need split them into two returns.

PR-URL: #26035
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: James M Snell <[email protected]>

lib/internal/process/main_thread_only.js

@@ -26,11 +26,9 @@ function wrapProcessMethods(binding) {
   }
 
   function umask(mask) {
-    if (mask === undefined) {
-      // Get the mask
-      return binding.umask(mask);
+    if (mask !== undefined) {
+      mask = validateMode(mask, 'mask');
     }
-    mask = validateMode(mask, 'mask');
     return binding.umask(mask);
   }