Adjust some comments in the StaticHeap test file.
#![feature(exact_size_is_empty)]
#![feature(trusted_len)]
-// In case you're wondering, the instances of `#[cfg_attr(all(windows, miri), ignore)]` in this
-// file above the `#[should_panic]` tests are there simply because Miri only supports catching
-// panics on Unix-like OSes and ignores `#[should_panic]` everywhere else, so without the
-// configuration attributes those tests just panic normally under Miri on Windows, which we don't
-// want.
-
// A note: This is literally the actual liballoc `BinaryHeap` test suite adapted for `StaticHeap`.
use core::iter::TrustedLen;
@@ -99,6 +93,7 @@ fn drain_sorted_collect() {
assert_eq!(sorted, staticvec![10, 9, 8, 7, 6, 5, 4, 3, 2, 2, 1, 1, 0]);
}
+// Miri cannot currently catch panics on Windows.
#[cfg_attr(all(windows, miri), ignore)]
#[test]
fn drain_sorted_leak() {
@@ -303,6 +298,8 @@ fn iter_rev_cloned_collect() {
// even if the order may not be correct.
//
// Destructors must be called exactly once per element.
+
+// Miri cannot currently catch panics on Windows.
#[cfg_attr(all(windows, miri), ignore)]
#[test]
fn panic_safe() {
