Don't time --no-execute

This made some output for non-execution.

See #977.

src/exec.cpp

@@ -957,7 +957,7 @@ bool exec_job(parser_t &parser, const shared_ptr<job_t> &j, const job_lineage_t
         // A false return tells the caller to remove the job from the list.
         return false;
     }
-    cleanup_t timer = push_timer(j->flags().has_time_prefix);
+    cleanup_t timer = push_timer(j->flags().has_time_prefix && !no_exec());
 
     // Get the deferred process, if any. We will have to remember its pipes.
     autoclose_pipes_t deferred_pipes;

src/parse_execution.cpp

@@ -1214,7 +1214,8 @@ end_execution_reason_t parse_execution_context_t::run_1_job(tnode_t<g::job> job_
     // is significantly faster.
     if (job_is_simple_block(job_node)) {
         tnode_t<g::optional_time> optional_time = job_node.child<0>();
-        cleanup_t timer = push_timer(optional_time.tag() == parse_optional_time_time);
+        // If no-exec has been given, there is nothing to time.
+        cleanup_t timer = push_timer(optional_time.tag() == parse_optional_time_time && !no_exec());
         tnode_t<g::variable_assignments> variable_assignments = job_node.child<1>();
         const block_t *block = nullptr;
         end_execution_reason_t result =

tests/checks/no-execute.fish

@@ -5,6 +5,11 @@ echo "type foo" | $fish -n
 echo $status
 #CHECK: 0
 
+# Test that it doesn't time non-execution.
+echo "time echo foo" | $fish -n
+echo $status
+#CHECK: 0
+
 # Test that it doesn't check globs.
 echo "echo /asfjidhfiusnlkxcnvklxcvlkmcxlv*" | $fish -n
 echo $status