Fix warnings on nightly.

tests/langtest_lua.rs

@@ -1,5 +1,3 @@
-#![feature(exit_status_error)]
-
 use inner::main;
 
 #[cfg(cargo_profile = "debug")]
@@ -49,7 +47,7 @@ mod inner {
         // more than one copy.
         let mut rsync_cmd = Command::new("rsync");
         rsync_cmd.args(["-a", YKLUA_SUBMODULE_PATH, yklua_tgt_dir.to_str().unwrap()]);
-        rsync_cmd.status().unwrap().exit_ok().unwrap();
+        assert!(rsync_cmd.status().unwrap().success());
 
         // cargo can sometimes run multiple builds in parallel. At the moment that's probably only
         // build scripts, though that doesn't seem to be precisely specified and might change in the

ykrt/src/lib.rs

@@ -3,7 +3,7 @@
 #![cfg_attr(test, feature(test))]
 #![feature(box_patterns)]
 #![feature(int_roundings)]
-#![feature(trim_prefix_suffix)]
+#![cfg_attr(test, feature(trim_prefix_suffix))]
 #![feature(thread_local)]
 #![allow(clippy::too_many_arguments)]
 #![allow(clippy::type_complexity)]

ykrt/src/mt.rs

@@ -339,18 +339,19 @@ impl MT {
                     mt.compiled_traces
                         .lock()
                         .insert(ctr.ctrid(), Arc::clone(&ctr));
-                    mt.stats.trace_compiled_ok();
                     match trace_start {
                         TraceStart::ControlPoint { hl } => {
                             let mut lk = hl.lock();
                             assert_matches!(lk.kind, HotLocationKind::Compiling(_));
                             lk.kind = HotLocationKind::Compiled(ctr);
+                            drop(lk);
                             mt.job_queue.notify_success(ctrid);
                         }
                         TraceStart::Guard { parent_ctr, gid } => {
                             parent_ctr.guard(gid).set_ctr(ctr, &parent_ctr, gid);
                         }
                     }
+                    mt.stats.trace_compiled_ok();
                 }
                 Err(e) => {
                     mt.stats.trace_compiled_err();