From 1c14fe33c07ee97b8ce31441ecb7fa0104080a5c Mon Sep 17 00:00:00 2001 From: wandalen Date: Tue, 14 May 2024 16:32:00 +0300 Subject: [PATCH 1/4] program_tools : evolve --- module/core/program_tools/src/program.rs | 38 +++++++++++------------- 1 file changed, 17 insertions(+), 21 deletions(-) diff --git a/module/core/program_tools/src/program.rs b/module/core/program_tools/src/program.rs index bb0c7aa51d..5ae707cf30 100644 --- a/module/core/program_tools/src/program.rs +++ b/module/core/program_tools/src/program.rs @@ -107,27 +107,23 @@ pub( crate ) mod private } } -// impl< IntoVariant > From< IntoVariant > for GetData -// where -// IntoVariant : Into< PathBuf >, -// { -// #[ inline ] -// fn from( src : IntoVariant ) -> Self -// { -// Self::FromStr( core::convert::Into::into( src ) ) -// } -// } -// -// impl< IntoVariant > From< IntoVariant > for GetData -// where -// IntoVariant : Into< String >, -// { -// #[ inline ] -// fn from( src : IntoVariant ) -> Self -// { -// Self::FromStr( core::convert::Into::into( src ) ) -// } -// } + impl From< PathBuf > for GetData + { + #[ inline ] + fn from( src : PathBuf ) -> Self + { + Self::FromFile( src ) + } + } + + impl From< String > for GetData + { + #[ inline ] + fn from( src : String ) -> Self + { + Self::FromString( src ) + } + } impl Default for GetData { From 9806451fc9c148fa4406160fb52c7dd7c509782d Mon Sep 17 00:00:00 2001 From: wandalen Date: Tue, 14 May 2024 16:51:21 +0300 Subject: [PATCH 2/4] derive_tools : from for variants : initial test --- .../tests/inc/from_inner_variants.rs | 27 +++++++++++++++++++ module/core/derive_tools/tests/inc/mod.rs | 2 ++ .../inc/only_test/from_inner_variants.rs | 15 +++++++++++ 3 files changed, 44 insertions(+) create mode 100644 module/core/derive_tools/tests/inc/from_inner_variants.rs create mode 100644 module/core/derive_tools/tests/inc/only_test/from_inner_variants.rs diff --git a/module/core/derive_tools/tests/inc/from_inner_variants.rs b/module/core/derive_tools/tests/inc/from_inner_variants.rs new file mode 100644 index 0000000000..4469adca5d --- /dev/null +++ b/module/core/derive_tools/tests/inc/from_inner_variants.rs @@ -0,0 +1,27 @@ + +#[ derive( Debug, PartialEq ) ] +pub enum GetData +{ + FromString( String ), + FromBin( &'static [ u8 ] ), +} + +impl From< String > for GetData +{ + #[ inline ] + fn from( src : String ) -> Self + { + Self::FromString( src ) + } +} + +impl From< &'static [ u8 ] > for GetData +{ + #[ inline ] + fn from( src : &'static [ u8 ] ) -> Self + { + Self::FromBin( src ) + } +} + +include!( "./only_test/from_inner_variants.rs" ); diff --git a/module/core/derive_tools/tests/inc/mod.rs b/module/core/derive_tools/tests/inc/mod.rs index babb3aca97..d23ab939e0 100644 --- a/module/core/derive_tools/tests/inc/mod.rs +++ b/module/core/derive_tools/tests/inc/mod.rs @@ -51,6 +51,8 @@ mod from_inner_multiple_named_test; mod from_inner_unit_test; #[ cfg( feature = "derive_from" ) ] mod from_inner_multiple_test; +#[ cfg( feature = "derive_from" ) ] +mod from_inner_variants; mod inner_from_manual_test; mod inner_from_named_manual_test; diff --git a/module/core/derive_tools/tests/inc/only_test/from_inner_variants.rs b/module/core/derive_tools/tests/inc/only_test/from_inner_variants.rs new file mode 100644 index 0000000000..417f5d6ca1 --- /dev/null +++ b/module/core/derive_tools/tests/inc/only_test/from_inner_variants.rs @@ -0,0 +1,15 @@ +use super::*; + +#[ test ] +fn from_inner_named() +{ + + let got : GetData = From::from( "abc".to_string() ); + let exp = GetData::FromString( "abc".to_string() ); + a_id!( got, exp ); + + let got : GetData = From::from( &b"abc"[ .. ] ); + let exp = GetData::FromBin( b"abc" ); + a_id!( got, exp ); + +} From 51daee2cbee40d8283ac4c23499ce5098594e613 Mon Sep 17 00:00:00 2001 From: wandalen Date: Tue, 14 May 2024 16:54:38 +0300 Subject: [PATCH 3/4] derive_tools : rid off alias FromInner --- .../core/derive_tools/tests/inc/all_test.rs | 2 +- .../inc/from_inner_multiple_named_test.rs | 2 +- .../tests/inc/from_inner_multiple_test.rs | 2 +- .../tests/inc/from_inner_named_test.rs | 2 +- .../derive_tools/tests/inc/from_inner_test.rs | 2 +- .../tests/inc/from_inner_unit_test.rs | 2 +- .../derive_tools/tests/inc/only_test/all.rs | 2 +- module/core/derive_tools_meta/src/lib.rs | 92 +++++++++---------- .../reflect_tools/tests/inc/only_test/all.rs | 2 +- .../src/hybrid_optimizer/sim_anneal.rs | 10 +- .../src/problems/sudoku/sudoku.rs | 38 ++++---- .../src/problems/traveling_salesman.rs | 4 +- 12 files changed, 80 insertions(+), 80 deletions(-) diff --git a/module/core/derive_tools/tests/inc/all_test.rs b/module/core/derive_tools/tests/inc/all_test.rs index a72ffa1741..8ca0db403f 100644 --- a/module/core/derive_tools/tests/inc/all_test.rs +++ b/module/core/derive_tools/tests/inc/all_test.rs @@ -1,6 +1,6 @@ use super::*; -#[ derive( Debug, Clone, Copy, PartialEq, /* the_module::Default,*/ the_module::FromInner, the_module::InnerFrom, the_module::Deref, the_module::DerefMut, the_module::AsRef, the_module::AsMut ) ] +#[ derive( Debug, Clone, Copy, PartialEq, /* the_module::Default,*/ the_module::From, the_module::InnerFrom, the_module::Deref, the_module::DerefMut, the_module::AsRef, the_module::AsMut ) ] // #[ default( value = false ) ] pub struct IsTransparent( bool ); diff --git a/module/core/derive_tools/tests/inc/from_inner_multiple_named_test.rs b/module/core/derive_tools/tests/inc/from_inner_multiple_named_test.rs index 436683a3b5..f12692c40a 100644 --- a/module/core/derive_tools/tests/inc/from_inner_multiple_named_test.rs +++ b/module/core/derive_tools/tests/inc/from_inner_multiple_named_test.rs @@ -1,6 +1,6 @@ use super::*; -#[ derive( Debug, PartialEq, Eq, the_module::FromInner ) ] +#[ derive( Debug, PartialEq, Eq, the_module::From ) ] struct StructNamedFields { a: i32, diff --git a/module/core/derive_tools/tests/inc/from_inner_multiple_test.rs b/module/core/derive_tools/tests/inc/from_inner_multiple_test.rs index dd18c948c9..db54cef54b 100644 --- a/module/core/derive_tools/tests/inc/from_inner_multiple_test.rs +++ b/module/core/derive_tools/tests/inc/from_inner_multiple_test.rs @@ -1,6 +1,6 @@ use super::*; -#[ derive( Debug, PartialEq, Eq, the_module::FromInner ) ] +#[ derive( Debug, PartialEq, Eq, the_module::From ) ] struct StructWithManyFields( i32, bool ); include!( "./only_test/from_inner_multiple.rs" ); diff --git a/module/core/derive_tools/tests/inc/from_inner_named_test.rs b/module/core/derive_tools/tests/inc/from_inner_named_test.rs index 0ea85ef088..c6a0f032d9 100644 --- a/module/core/derive_tools/tests/inc/from_inner_named_test.rs +++ b/module/core/derive_tools/tests/inc/from_inner_named_test.rs @@ -1,6 +1,6 @@ use super::*; -#[ derive( Debug, PartialEq, Eq, the_module::FromInner ) ] +#[ derive( Debug, PartialEq, Eq, the_module::From ) ] struct MyStruct { a: i32, diff --git a/module/core/derive_tools/tests/inc/from_inner_test.rs b/module/core/derive_tools/tests/inc/from_inner_test.rs index 4848773fde..98890e6c2e 100644 --- a/module/core/derive_tools/tests/inc/from_inner_test.rs +++ b/module/core/derive_tools/tests/inc/from_inner_test.rs @@ -3,7 +3,7 @@ use super::*; // use diagnostics_tools::prelude::*; // use derives::*; -#[ derive( Debug, Clone, Copy, PartialEq, the_module::FromInner ) ] +#[ derive( Debug, Clone, Copy, PartialEq, the_module::From ) ] pub struct IsTransparent( bool ); // include!( "./manual/basic.rs" ); diff --git a/module/core/derive_tools/tests/inc/from_inner_unit_test.rs b/module/core/derive_tools/tests/inc/from_inner_unit_test.rs index 2aa637a05b..e551263159 100644 --- a/module/core/derive_tools/tests/inc/from_inner_unit_test.rs +++ b/module/core/derive_tools/tests/inc/from_inner_unit_test.rs @@ -1,6 +1,6 @@ use super::*; -#[ derive( Debug, Clone, Copy, PartialEq, the_module::FromInner ) ] +#[ derive( Debug, Clone, Copy, PartialEq, the_module::From ) ] struct UnitStruct; include!( "./only_test/from_inner_unit.rs" ); diff --git a/module/core/derive_tools/tests/inc/only_test/all.rs b/module/core/derive_tools/tests/inc/only_test/all.rs index 9708a9f8cf..5fe5831993 100644 --- a/module/core/derive_tools/tests/inc/only_test/all.rs +++ b/module/core/derive_tools/tests/inc/only_test/all.rs @@ -7,7 +7,7 @@ fn basic_test() let exp = IsTransparent( true ); a_id!( got, exp ); - // FromInner + // From let got = IsTransparent::from( true ); let exp = IsTransparent( true ); diff --git a/module/core/derive_tools_meta/src/lib.rs b/module/core/derive_tools_meta/src/lib.rs index 97cb37042f..338eb397f6 100644 --- a/module/core/derive_tools_meta/src/lib.rs +++ b/module/core/derive_tools_meta/src/lib.rs @@ -83,52 +83,52 @@ pub fn from( input : proc_macro::TokenStream ) -> proc_macro::TokenStream } } -/// -/// Alias for derive `From`. Provides an automatic `From` implementation for struct wrapping a single value. -/// -/// This macro simplifies the conversion of an inner type to an outer struct type -/// when the outer type is a simple wrapper around the inner type. -/// -/// ## Example Usage -/// -/// Instead of manually implementing `From< bool >` for `IsTransparent`: -/// -/// ```rust -/// pub struct IsTransparent( bool ); -/// -/// impl From< bool > for IsTransparent -/// { -/// #[ inline( always ) ] -/// fn from( src : bool ) -> Self -/// { -/// Self( src ) -/// } -/// } -/// ``` -/// -/// Use `#[ derive( FromInner ) ]` to automatically generate the implementation: -/// -/// ```rust -/// # use derive_tools_meta::*; -/// #[ derive( FromInner ) ] -/// pub struct IsTransparent( bool ); -/// ``` -/// -/// The macro facilitates the conversion without additional boilerplate code. -/// - -#[ cfg( feature = "enabled" ) ] -#[ cfg( feature = "derive_from" ) ] -#[ proc_macro_derive( FromInner ) ] -pub fn from_inner( input : proc_macro::TokenStream ) -> proc_macro::TokenStream -{ - let result = derive::from::from( input ); - match result - { - Ok( stream ) => stream.into(), - Err( err ) => err.to_compile_error().into(), - } -} +// /// +// /// Alias for derive `From`. Provides an automatic `From` implementation for struct wrapping a single value. +// /// +// /// This macro simplifies the conversion of an inner type to an outer struct type +// /// when the outer type is a simple wrapper around the inner type. +// /// +// /// ## Example Usage +// /// +// /// Instead of manually implementing `From< bool >` for `IsTransparent`: +// /// +// /// ```rust +// /// pub struct IsTransparent( bool ); +// /// +// /// impl From< bool > for IsTransparent +// /// { +// /// #[ inline( always ) ] +// /// fn from( src : bool ) -> Self +// /// { +// /// Self( src ) +// /// } +// /// } +// /// ``` +// /// +// /// Use `#[ derive( FromInner ) ]` to automatically generate the implementation: +// /// +// /// ```rust +// /// # use derive_tools_meta::*; +// /// #[ derive( FromInner ) ] +// /// pub struct IsTransparent( bool ); +// /// ``` +// /// +// /// The macro facilitates the conversion without additional boilerplate code. +// /// +// +// #[ cfg( feature = "enabled" ) ] +// #[ cfg( feature = "derive_from" ) ] +// #[ proc_macro_derive( FromInner ) ] +// pub fn from_inner( input : proc_macro::TokenStream ) -> proc_macro::TokenStream +// { +// let result = derive::from::from( input ); +// match result +// { +// Ok( stream ) => stream.into(), +// Err( err ) => err.to_compile_error().into(), +// } +// } /// /// Derive macro to implement From converting outer type into inner when-ever it's possible to do automatically. diff --git a/module/core/reflect_tools/tests/inc/only_test/all.rs b/module/core/reflect_tools/tests/inc/only_test/all.rs index 9708a9f8cf..5fe5831993 100644 --- a/module/core/reflect_tools/tests/inc/only_test/all.rs +++ b/module/core/reflect_tools/tests/inc/only_test/all.rs @@ -7,7 +7,7 @@ fn basic_test() let exp = IsTransparent( true ); a_id!( got, exp ); - // FromInner + // From let got = IsTransparent::from( true ); let exp = IsTransparent( true ); diff --git a/module/move/optimization_tools/src/hybrid_optimizer/sim_anneal.rs b/module/move/optimization_tools/src/hybrid_optimizer/sim_anneal.rs index c176729441..f7d4c5743a 100644 --- a/module/move/optimization_tools/src/hybrid_optimizer/sim_anneal.rs +++ b/module/move/optimization_tools/src/hybrid_optimizer/sim_anneal.rs @@ -1,8 +1,8 @@ //! Implementation of Simulated Annealing for Hybrid Optimizer. -use derive_tools::{ FromInner, InnerFrom, exposed::Display }; +use derive_tools::{ From, InnerFrom, exposed::Display }; /// Represents temperature of SA process. -#[ derive( Default, Debug, Display, Clone, Copy, PartialEq, PartialOrd, FromInner, InnerFrom ) ] +#[ derive( Default, Debug, Display, Clone, Copy, PartialEq, PartialOrd, From, InnerFrom ) ] pub struct Temperature( f64 ); impl Temperature @@ -27,7 +27,7 @@ impl From< f32 > for Temperature // use derive_tools::{ Add, Sub, Mul, Div, AddAssign, SubAssign, MulAssign, DivAssign }; /// Struct that represents coefficient to change temperature value. -#[ derive( Debug, Display, Clone, Copy, PartialEq, PartialOrd, FromInner, InnerFrom ) ] +#[ derive( Debug, Display, Clone, Copy, PartialEq, PartialOrd, From, InnerFrom ) ] // #[ derive( Add, Sub, Mul, Div, AddAssign, SubAssign, MulAssign, DivAssign ) ] pub struct TemperatureFactor( pub f64 ); @@ -83,12 +83,12 @@ pub struct LinearTempSchedule impl TemperatureSchedule for LinearTempSchedule { - fn calculate_next_temp( &self, prev_temp : Temperature ) -> Temperature + fn calculate_next_temp( &self, prev_temp : Temperature ) -> Temperature { Temperature::from( prev_temp.unwrap() * self.coefficient.unwrap() + self.constant.unwrap() ) } - fn reset_temperature( &self, prev_temp : Temperature ) -> Temperature + fn reset_temperature( &self, prev_temp : Temperature ) -> Temperature { Temperature( prev_temp.unwrap() + self.reset_increase_value.unwrap() ) } diff --git a/module/move/optimization_tools/src/problems/sudoku/sudoku.rs b/module/move/optimization_tools/src/problems/sudoku/sudoku.rs index b016fa4cda..eac6a5dbda 100644 --- a/module/move/optimization_tools/src/problems/sudoku/sudoku.rs +++ b/module/move/optimization_tools/src/problems/sudoku/sudoku.rs @@ -4,7 +4,7 @@ use std::collections::HashSet; use crate::hybrid_optimizer::*; use crate::problems::sudoku::*; -use derive_tools::{ FromInner, InnerFrom, exposed::Display }; +use derive_tools::{ From, InnerFrom, exposed::Display }; use deterministic_rand::{ Hrng, Rng, seq::SliceRandom }; use iter_tools::Itertools; @@ -72,7 +72,7 @@ pub fn cells_pair_random_in_block( initial : &Board, block : BlockIndex, hrng : } /// Represents number of errors in sudoku board. -#[ derive( Default, Debug, Display, Clone, Copy, PartialEq, Eq, Ord, PartialOrd, Hash, FromInner, InnerFrom ) ] +#[ derive( Default, Debug, Display, Clone, Copy, PartialEq, Eq, Ord, PartialOrd, Hash, From, InnerFrom ) ] pub struct SudokuCost( usize ); // xxx : derive, please @@ -113,13 +113,13 @@ impl Individual for SudokuPerson { true } - else + else { false } } - fn fitness( &self ) -> usize + fn fitness( &self ) -> usize { self.cost.into() } @@ -153,7 +153,7 @@ impl SudokuPerson { let old_cross_error = self.board.cross_error( mutagen.cell1 ) + self.board.cross_error( mutagen.cell2 ); - + log::trace!( "cells_swap( {:?}, {:?} )", mutagen.cell1, mutagen.cell2 ); self.board.cells_swap( mutagen.cell1, mutagen.cell2 ); self.cost = SudokuCost( self.cost.unwrap() - old_cross_error ) ; @@ -174,8 +174,8 @@ impl SudokuPerson pub fn mutagen( &self, initial : &Board, hrng : Hrng ) -> SudokuMutagen { let mutagen; - loop - { + loop + { let rng_ref = hrng.rng_ref(); let mut rng = rng_ref.lock().unwrap(); let block : BlockIndex = rng.gen(); @@ -191,7 +191,7 @@ impl SudokuPerson } /// Represents single change(mutation) which contains indeces of two swapped cells. It is used to generate new state of the board for sudoku solving process. -#[ derive( PartialEq, Eq, Clone, Debug, FromInner, InnerFrom ) ] +#[ derive( PartialEq, Eq, Clone, Debug, From, InnerFrom ) ] pub struct SudokuMutagen { /// Index of cell swapped in mutation. @@ -221,12 +221,12 @@ impl InitialProblem for SudokuInitial { type Person = SudokuPerson; - fn get_random_person( &self, hrng : Hrng ) -> SudokuPerson + fn get_random_person( &self, hrng : Hrng ) -> SudokuPerson { SudokuPerson::new( &self.board, hrng.clone() ) } - fn evaluate( &self, person : &SudokuPerson ) -> f64 + fn evaluate( &self, person : &SudokuPerson ) -> f64 { person.board.total_error() as f64 } @@ -241,11 +241,11 @@ impl MutationOperator for RandomPairInBlockMutation type Person = SudokuPerson; type Problem = SudokuInitial; - fn mutate( &self, hrng : Hrng, person : &mut Self::Person, context : &Self::Problem ) + fn mutate( &self, hrng : Hrng, person : &mut Self::Person, context : &Self::Problem ) { let mutagen : SudokuMutagen = - loop - { + loop + { let rng_ref = hrng.rng_ref(); let mut rng = rng_ref.lock().unwrap(); let block : BlockIndex = rng.gen(); @@ -257,7 +257,7 @@ impl MutationOperator for RandomPairInBlockMutation }.into(); let old_cross_error = person.board.cross_error( mutagen.cell1 ) + person.board.cross_error( mutagen.cell2 ); - + log::trace!( "cells_swap( {:?}, {:?} )", mutagen.cell1, mutagen.cell2 ); person.board.cells_swap( mutagen.cell1, mutagen.cell2 ); person.cost = SudokuCost( person.cost.unwrap() - old_cross_error ); @@ -301,7 +301,7 @@ impl CrossoverOperator for MultiplePointsBlockCrossover child_storage[ usize::from( cell_index ) ] = parent_block[ index ]; } } - else + else { let parent_block = parent2.board.block( i ).collect_vec(); let cells = parent2.board.block_cells( i ); @@ -311,7 +311,7 @@ impl CrossoverOperator for MultiplePointsBlockCrossover } } } - + let child = SudokuPerson::with_board( Board::new( child_storage ) ); child } @@ -324,7 +324,7 @@ pub struct BestRowsColumnsCrossover; impl CrossoverOperator for BestRowsColumnsCrossover { type Person = < SudokuInitial as InitialProblem >::Person; - + fn crossover( &self, _hrng : Hrng, parent1 : &Self::Person, parent2 : &Self::Person ) -> Self::Person { let mut rows_costs = vec![ Vec::new(); 2 ]; @@ -393,7 +393,7 @@ impl CrossoverOperator for BestRowsColumnsCrossover child2_storage[ usize::from( cell_index ) ] = parent_block[ index ]; } } - else + else { let parent_block = parent2.board.block( BlockIndex::from( ( j as u8, i as u8 ) ) ).collect_vec(); let cells = parent2.board.block_cells( BlockIndex::from( ( j as u8, i as u8 ) ) ); @@ -411,6 +411,6 @@ impl CrossoverOperator for BestRowsColumnsCrossover .unwrap() ; - SudokuPerson::with_board( min_board ) + SudokuPerson::with_board( min_board ) } } diff --git a/module/move/optimization_tools/src/problems/traveling_salesman.rs b/module/move/optimization_tools/src/problems/traveling_salesman.rs index 2c3e5bb9a1..2eab112158 100644 --- a/module/move/optimization_tools/src/problems/traveling_salesman.rs +++ b/module/move/optimization_tools/src/problems/traveling_salesman.rs @@ -17,7 +17,7 @@ use std::collections::HashMap; use crate::hybrid_optimizer::*; -use derive_tools::{ FromInner, InnerFrom }; +use derive_tools::{ From, InnerFrom }; use deterministic_rand::{ Hrng, seq::{ SliceRandom, IteratorRandom } }; use iter_tools::Itertools; @@ -89,7 +89,7 @@ pub struct Node< T > pub struct NodeIndex( pub usize ); /// Weight of graph edge. -#[ derive( Debug, FromInner, InnerFrom, Clone, Copy ) ] +#[ derive( Debug, From, InnerFrom, Clone, Copy ) ] pub struct EdgeWeight( pub f64 ); /// Edge for undirected weighted graph. From 33ce1027eaddc73a178098877a1c7c59305c0c8b Mon Sep 17 00:00:00 2001 From: wandalen Date: Tue, 14 May 2024 18:21:42 +0300 Subject: [PATCH 4/4] derive_tools : tasks --- module/core/derive_tools/examples/derive_tools_trivial.rs | 2 +- module/core/derive_tools/src/wtools.rs | 2 +- module/core/derive_tools/tests/inc/basic_test.rs | 2 +- module/core/derive_tools/tests/inc/mod.rs | 2 +- module/core/derive_tools_meta/Cargo.toml | 2 +- module/core/derive_tools_meta/src/derive/from.rs | 2 +- module/core/derive_tools_meta/src/lib.rs | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/module/core/derive_tools/examples/derive_tools_trivial.rs b/module/core/derive_tools/examples/derive_tools_trivial.rs index ff402f3c86..1d338ee621 100644 --- a/module/core/derive_tools/examples/derive_tools_trivial.rs +++ b/module/core/derive_tools/examples/derive_tools_trivial.rs @@ -1,4 +1,4 @@ -//! qqq : write proper description +//! qqq : for Petro : write proper description fn main() { diff --git a/module/core/derive_tools/src/wtools.rs b/module/core/derive_tools/src/wtools.rs index 97e8a54e95..64136ce040 100644 --- a/module/core/derive_tools/src/wtools.rs +++ b/module/core/derive_tools/src/wtools.rs @@ -2,7 +2,7 @@ //! Types, which are extension of std. //! -// qqq : xxx : rid off the file +// qqq : for Petro : xxx : rid off the file /// Internal namespace. pub( crate ) mod private diff --git a/module/core/derive_tools/tests/inc/basic_test.rs b/module/core/derive_tools/tests/inc/basic_test.rs index f83df41abe..0e1d8ab1f1 100644 --- a/module/core/derive_tools/tests/inc/basic_test.rs +++ b/module/core/derive_tools/tests/inc/basic_test.rs @@ -12,7 +12,7 @@ tests_impls! { use the_module::*; - // xxx : qqq : make it working + // xxx : qqq : for Petro : make it working #[ derive( From, InnerFrom, Display, FromStr, PartialEq, Debug ) ] #[ display( "{a}-{b}" ) ] struct Struct1 diff --git a/module/core/derive_tools/tests/inc/mod.rs b/module/core/derive_tools/tests/inc/mod.rs index d23ab939e0..dd16de6655 100644 --- a/module/core/derive_tools/tests/inc/mod.rs +++ b/module/core/derive_tools/tests/inc/mod.rs @@ -70,7 +70,7 @@ mod inner_from_unit_test; #[ cfg( feature = "derive_inner_from" ) ] mod inner_from_multiple_test; -// qqq : xxx : fix +// qqq : for Petro : xxx : fix // #[ cfg( all( feature = "type_variadic_from" ) ) ] // mod variadic_from_manual_test; // diff --git a/module/core/derive_tools_meta/Cargo.toml b/module/core/derive_tools_meta/Cargo.toml index 5eba0b3302..38c803eed4 100644 --- a/module/core/derive_tools_meta/Cargo.toml +++ b/module/core/derive_tools_meta/Cargo.toml @@ -63,7 +63,7 @@ derive_variadic_from = [] [dependencies] macro_tools = { workspace = true, features = [ "full" ] } iter_tools = { workspace = true, features = [ "full" ] } -# xxx : qqq : optimize features set +# xxx : qqq : for Petro : optimize features set [dev-dependencies] test_tools = { workspace = true } diff --git a/module/core/derive_tools_meta/src/derive/from.rs b/module/core/derive_tools_meta/src/derive/from.rs index 43b5e727aa..1a415eed14 100644 --- a/module/core/derive_tools_meta/src/derive/from.rs +++ b/module/core/derive_tools_meta/src/derive/from.rs @@ -71,7 +71,7 @@ fn generate_from_single_field } } -// qqq : document, add example of generated code +// qqq : for Petro : document, add example of generated code fn generate_from_multiple_fields_named ( field_types : &Vec< &syn::Type >, diff --git a/module/core/derive_tools_meta/src/lib.rs b/module/core/derive_tools_meta/src/lib.rs index 338eb397f6..de4f5f2527 100644 --- a/module/core/derive_tools_meta/src/lib.rs +++ b/module/core/derive_tools_meta/src/lib.rs @@ -374,7 +374,7 @@ pub fn as_mut( input : proc_macro::TokenStream ) -> proc_macro::TokenStream /// /// ``` -// qqq : xxx : why no run/ignore? fix +// qqq : for Petro : xxx : why no run/ignore? fix #[ cfg( feature = "enabled" ) ] #[ cfg( feature = "derive_variadic_from" ) ]