Browse Source

lang: Fix redundant split() (#960)

wan-nyan-wan 3 years ago
parent
commit
ddd5ad7a48
1 changed files with 4 additions and 2 deletions
  1. 4 2
      lang/attribute/account/src/lib.rs

+ 4 - 2
lang/attribute/account/src/lib.rs

@@ -61,10 +61,12 @@ pub fn account(
 ) -> proc_macro::TokenStream {
 ) -> proc_macro::TokenStream {
     let mut namespace = "".to_string();
     let mut namespace = "".to_string();
     let mut is_zero_copy = false;
     let mut is_zero_copy = false;
-    if args.to_string().split(',').count() > 2 {
+    let args_str = args.to_string();
+    let args: Vec<&str> = args_str.split(',').collect();
+    if args.len() > 2 {
         panic!("Only two args are allowed to the account attribute.")
         panic!("Only two args are allowed to the account attribute.")
     }
     }
-    for arg in args.to_string().split(',') {
+    for arg in args {
         let ns = arg
         let ns = arg
             .to_string()
             .to_string()
             .replace("\"", "")
             .replace("\"", "")