Merge pull request #623 from nicholasbishop/bishop-alloc-cleanup

Minor alloc-related cleanups

Author: phip1611

uefi-test-runner/src/boot/memory.rs

@@ -1,6 +1,6 @@
 use uefi::table::boot::{AllocateType, BootServices, MemoryType};
 
-use crate::alloc::vec::Vec;
+use alloc::vec::Vec;
 
 pub fn test(bt: &BootServices) {
     info!("Testing memory functions");

uefi/src/data_types/owned_strs.rs

@@ -1,9 +1,9 @@
 use super::chars::{Char16, NUL_16};
 use super::strs::{CStr16, FromSliceWithNulError};
-use crate::alloc::vec::Vec;
 use crate::data_types::strs::EqStrUntilNul;
 use crate::data_types::UnalignedSlice;
 use crate::polyfill::vec_into_raw_parts;
+use alloc::vec::Vec;
 use core::fmt;
 use core::ops;
 
@@ -155,8 +155,8 @@ impl<StrType: AsRef<str> + ?Sized> EqStrUntilNul<StrType> for CString16 {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::alloc::string::String;
-    use crate::alloc::vec;
+    use alloc::string::String;
+    use alloc::vec;
 
     #[test]
     fn test_cstring16_from_str() {

uefi/src/data_types/strs.rs

@@ -478,7 +478,7 @@ where
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::alloc::string::String;
+    use alloc::string::String;
     use uefi_macros::{cstr16, cstr8};
 
     // Tests if our CStr8 type can be constructed from a valid core::ffi::CStr

uefi/src/data_types/unaligned_slice.rs

@@ -3,7 +3,7 @@ use core::marker::PhantomData;
 use core::mem::MaybeUninit;
 
 #[cfg(feature = "alloc")]
-use crate::alloc::vec::Vec;
+use alloc::vec::Vec;
 
 /// Slice backed by a potentially-unaligned pointer.
 ///

uefi/src/lib.rs

@@ -68,8 +68,6 @@
 #![deny(clippy::all)]
 #![deny(clippy::must_use_candidate)]
 
-// Enable once we use vec![] or similar
-// #[cfg_attr(feature = "alloc", macro_use)]
 #[cfg(feature = "alloc")]
 extern crate alloc;
 

uefi/src/proto/media/file/info.rs

@@ -398,10 +398,10 @@ impl FileProtocolInfo for FileSystemVolumeLabel {}
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::alloc::vec;
     use crate::table::runtime::TimeParams;
     use crate::table::runtime::{Daylight, Time};
     use crate::CString16;
+    use alloc::vec;
 
     fn validate_layout<T: InfoInternal + ?Sized>(info: &T, name: &[Char16]) {
         // Check the hardcoded struct alignment.