fixes

Author: Wandalen

Cargo.toml

@@ -31,8 +31,6 @@ exclude = [
   "module/alias/wtest",
   "module/core/meta_tools",
   "module/core/for_each",
-  "module/core/reflect_tools",
-  "module/core/format_tools",
   "step",
 ]
 # default-members = [ "module/core/wtools" ]

module/core/collection_tools/src/lib.rs

@@ -46,6 +46,7 @@ pub mod own {
 
   #[ doc( inline ) ]
   #[ allow( clippy::useless_attribute, clippy::pub_use ) ]
+  #[cfg(any(feature = "use_alloc", not(feature = "no_std")))]
   pub use super::collection::own::*;
 }
 
@@ -61,6 +62,7 @@ pub mod orphan {
 
   #[ doc( inline ) ]
   #[ allow( clippy::useless_attribute, clippy::pub_use ) ]
+  #[cfg(any(feature = "use_alloc", not(feature = "no_std")))]
   pub use collection::orphan::*;
 }
 
@@ -73,10 +75,12 @@ pub mod exposed {
 
   #[ doc( inline ) ]
   #[ allow( clippy::useless_attribute, clippy::pub_use ) ]
+  #[cfg(any(feature = "use_alloc", not(feature = "no_std")))]
   pub use prelude::*;
 
   #[ doc( inline ) ]
   #[ allow( clippy::useless_attribute, clippy::pub_use ) ]
+  #[cfg(any(feature = "use_alloc", not(feature = "no_std")))]
   pub use collection::exposed::*;
 }
 
@@ -92,6 +96,13 @@ pub mod prelude {
   pub use collection::prelude::*;
 }
 
+/// Empty prelude for no_std configurations
+#[ cfg( feature = "enabled" ) ]
+#[cfg(all(feature = "no_std", not(feature = "use_alloc")))]
+#[ allow( unused_imports ) ]
+pub mod prelude {
+}
+
 // pub use own::collection as xxx;
 // pub use hmap as xxx;
 // pub use own::HashMap as xxx;

module/core/collection_tools/tests/inc/llist.rs

@@ -49,6 +49,7 @@ fn into_constructor() {
 #[ test ]
 fn iters() {
   struct MyContainer {
+    #[allow(clippy::linkedlist)]
     entries: the_module::LinkedList<i32>,
   }
 

module/core/collection_tools/tests/inc/vec.rs

@@ -3,18 +3,14 @@ use super::*;
 #[ test ]
 #[cfg(any(feature = "use_alloc", not(feature = "no_std")))]
 fn reexport() {
-  let mut vec1: the_module::Vec< i32 > = the_module::Vec::new();
-  vec1.push(1);
-  vec1.push(2);
+  let vec1: the_module::Vec< i32 > = the_module::vec![ 1, 2 ];
   let got = *vec1.first().unwrap();
   assert_eq!(got, 1);
   let got = *vec1.last().unwrap();
   assert_eq!(got, 2);
 
   use std::vec::Vec as DynList;
-  let mut vec2: DynList<i32> = DynList::new();
-  vec2.push(1);
-  vec2.push(2);
+  let vec2: DynList<i32> = DynList::from([ 1, 2 ]);
   let got = *vec2.first().unwrap();
   assert_eq!(got, 1);
   let got = *vec2.last().unwrap();
@@ -33,9 +29,7 @@ fn constructor() {
 
   // test.case( "multiple entry" );
   let got = the_module::vec! { 3, 13 };
-  let mut exp = the_module::Vec::new();
-  exp.push(3);
-  exp.push(13);
+  let exp = the_module::vec![ 3, 13 ];
   assert_eq!(got, exp);
 
   let _got = the_module::vec!("b");
@@ -53,9 +47,7 @@ fn into_constructor() {
 
   // test.case( "multiple entry" );
   let got: the_module::Vec< i32 > = the_module::into_vec! { 3, 13 };
-  let mut exp = the_module::Vec::new();
-  exp.push(3);
-  exp.push(13);
+  let exp = the_module::vec![ 3, 13 ];
   assert_eq!(got, exp);
 
   let _got: Vec< &str > = the_module::into_vec!("b");

module/core/format_tools/src/format/to_string/aref.rs

@@ -7,6 +7,7 @@ use core::ops::{ Deref };
 
 /// Reference wrapper to make into string conversion with fallback.
 #[ allow( missing_debug_implementations ) ]
+#[ allow( dead_code ) ]
 #[ repr( transparent ) ]
 pub struct Ref< 'a, T, How >
 ( pub Ref2< 'a, T, How > )

module/core/format_tools/src/lib.rs

@@ -3,6 +3,56 @@
 #![ doc( html_root_url = "https://docs.rs/reflect_tools/latest/reflect_tools/" ) ]
 #![ cfg_attr( doc, doc = include_str!( concat!( env!( "CARGO_MANIFEST_DIR" ), "/", "readme.md" ) ) ) ]
 #![ cfg_attr( not( doc ), doc = "Formatting utilities" ) ]
+#![ allow( clippy::similar_names ) ]
+#![ allow( clippy::double_parens ) ]
+#![ allow( clippy::empty_line_after_doc_comments ) ]
+#![ allow( clippy::redundant_else ) ]
+#![ allow( clippy::single_match_else ) ]
+#![ allow( clippy::needless_late_init ) ]
+#![ allow( clippy::match_same_arms ) ]
+#![ allow( clippy::implicit_clone ) ]
+#![ allow( clippy::unnecessary_wraps ) ]
+#![ allow( clippy::explicit_iter_loop ) ]
+#![ allow( clippy::elidable_lifetime_names ) ]
+#![ allow( clippy::needless_borrow ) ]
+#![ allow( clippy::std_instead_of_core ) ]
+#![ allow( clippy::doc_lazy_continuation ) ]
+#![ allow( clippy::cast_possible_truncation ) ]
+#![ allow( clippy::cast_sign_loss ) ]
+#![ allow( clippy::must_use_candidate ) ]
+#![ allow( clippy::unreadable_literal ) ]
+#![ allow( clippy::type_complexity ) ]
+#![ allow( clippy::default_trait_access ) ]
+#![ allow( clippy::missing_errors_doc ) ]
+#![ allow( clippy::manual_string_new ) ]
+#![ allow( clippy::explicit_counter_loop ) ]
+#![ allow( clippy::uninlined_format_args ) ]
+#![ allow( clippy::manual_map ) ]
+#![ allow( clippy::doc_markdown ) ]
+#![ allow( clippy::extra_unused_lifetimes ) ]
+#![ allow( clippy::unnecessary_cast ) ]
+#![ allow( clippy::redundant_closure ) ]
+#![ allow( clippy::needless_borrows_for_generic_args ) ]
+#![ allow( clippy::derivable_impls ) ]
+#![ allow( clippy::write_with_newline ) ]
+#![ allow( clippy::bool_to_int_with_if ) ]
+#![ allow( clippy::redundant_static_lifetimes ) ]
+#![ allow( clippy::inconsistent_struct_constructor ) ]
+#![ allow( clippy::len_zero ) ]
+#![ allow( clippy::needless_as_bytes ) ]
+#![ allow( clippy::struct_field_names ) ]
+#![ allow( clippy::unnecessary_semicolon ) ]
+#![ allow( clippy::match_bool ) ]
+#![ allow( clippy::implicit_hasher ) ]
+#![ allow( clippy::map_identity ) ]
+#![ allow( clippy::manual_repeat_n ) ]
+#![ allow( clippy::too_many_lines ) ]
+#![ allow( clippy::needless_pass_by_value ) ]
+#![ allow( clippy::collapsible_else_if ) ]
+#![ allow( clippy::needless_return ) ]
+#![ allow( clippy::needless_raw_string_hashes ) ]
+#![ allow( clippy::ref_option ) ]
+#![ allow( clippy::owned_cow ) ]
 
 #[ cfg( feature = "enabled" ) ]
 pub mod format;

module/core/format_tools/tests/tests.rs

@@ -2,6 +2,19 @@
 
 // #![ feature( trace_macros ) ]
 #![ allow( unused_imports ) ]
+#![ allow( clippy::unreadable_literal ) ]
+#![ allow( clippy::needless_raw_string_hashes ) ]
+#![ allow( clippy::default_trait_access ) ]
+#![ allow( clippy::uninlined_format_args ) ]
+#![ allow( clippy::ref_option ) ]
+#![ allow( clippy::useless_conversion ) ]
+#![ allow( clippy::owned_cow ) ]
+#![ allow( clippy::type_complexity ) ]
+#![ allow( clippy::elidable_lifetime_names ) ]
+#![ allow( clippy::redundant_closure ) ]
+#![ allow( clippy::println_empty_string ) ]
+#![ allow( clippy::field_reassign_with_default ) ]
+#![ allow( clippy::never_loop ) ]
 
 use format_tools as the_module;
 use test_tools::exposed::*;

module/core/pth/src/path.rs

@@ -208,6 +208,7 @@ mod private
     #[ cfg( feature = "no_std" ) ]
     extern crate alloc;
     #[ cfg( feature = "no_std" ) ]
+    #[ allow( unused_imports ) ]
     use alloc::string::ToString;
 
     // println!( "a" );
@@ -330,11 +331,13 @@ mod private
   /// # Panics
   /// qqq: doc
   // qqq : make macro paths_join!( ... )
-  pub fn iter_join< 'a ,I, P >( paths : I ) -> PathBuf
+  pub fn iter_join< 'a ,I, P >( paths : I ) -> std::path::PathBuf
   where
     I : Iterator< Item = P >,
     P : TryIntoCowPath< 'a >,
   {
+    #[ allow( unused_imports ) ]
+    use std::path::PathBuf;
     #[ cfg( feature = "no_std" ) ]
     extern crate alloc;
     #[ cfg( feature = "no_std" ) ]

module/core/pth/src/path/absolute_path.rs

@@ -5,6 +5,7 @@ mod private
   use std::
   {
     path::{ Path, PathBuf },
+    borrow::Cow,
     io,
   };
   use core::

module/core/pth/src/path/canonical_path.rs

@@ -5,7 +5,7 @@ mod private
 
   use std::
   {
-    // borrow::Cow,
+    borrow::Cow,
     path::{ Path, PathBuf },
     io,
   };