chore: minor cleanup (#32)

* chore: minor cleanup

Signed-off-by: Henry Schreiner <[email protected]>

* Update table.rs

---------

Signed-off-by: Henry Schreiner <[email protected]>

Author: henryiii

.gitignore

@@ -10,3 +10,4 @@ dist
 /src/pyproject_fmt_rust/_lib.abi3*
 /tarpaulin-report.html
 /build_rs_cov.profraw
+/.cargo/config.toml

Cargo.lock

@@ -3,7 +3,7 @@ use crate::helpers::pep508::{format_requirement, get_canonic_requirement_name};
 use crate::helpers::table::{for_entries, reorder_table_keys, Tables};
 
 pub fn fix(tables: &mut Tables, keep_full_version: bool) {
-    let table_element = tables.get(&String::from("build-system"));
+    let table_element = tables.get("build-system");
     if table_element.is_none() {
         return;
     }

rust/src/build_system.rs

@@ -186,7 +186,7 @@ pub fn reorder_table_keys(table: &mut RefMut<Vec<SyntaxElement>>, order: &[&str]
     table.splice(0..size, to_insert);
 }
 
-fn load_keys(table: &RefMut<Vec<SyntaxElement>>) -> (HashMap<String, usize>, Vec<Vec<SyntaxElement>>) {
+fn load_keys(table: &[SyntaxElement]) -> (HashMap<String, usize>, Vec<Vec<SyntaxElement>>) {
     let mut key_to_pos = HashMap::<String, usize>::new();
     let mut key_set = Vec::<Vec<SyntaxElement>>::new();
     let entry_set = RefCell::new(Vec::<SyntaxElement>::new());
@@ -200,7 +200,7 @@ fn load_keys(table: &RefMut<Vec<SyntaxElement>>) -> (HashMap<String, usize>, Vec
     };
     let mut key = String::new();
     let mut cutoff = false;
-    for element in table.iter() {
+    for element in table {
         let kind = element.kind();
         if kind == ENTRY {
             if cutoff {
@@ -238,12 +238,12 @@ pub fn get_table_name(entry: &SyntaxElement) -> String {
     String::new()
 }
 
-pub fn for_entries<F>(table: &RefMut<Vec<SyntaxElement>>, f: &mut F)
+pub fn for_entries<F>(table: &[SyntaxElement], f: &mut F)
 where
     F: FnMut(String, &SyntaxNode),
 {
     let mut key = String::new();
-    for table_entry in table.iter() {
+    for table_entry in table {
         if table_entry.kind() == ENTRY {
             for entry in table_entry.as_node().unwrap().children_with_tokens() {
                 if entry.kind() == KEY {

rust/src/helpers/table.rs

@@ -21,7 +21,7 @@ pub fn fix(
     min_supported_python: (u8, u8),
 ) {
     collapse_sub_tables(tables, "project");
-    let table_element = tables.get(&String::from("project"));
+    let table_element = tables.get("project");
     if table_element.is_none() {
         return;
     }
@@ -291,7 +291,7 @@ fn generate_classifiers_to_entry(
 type MaxMinPythonWithClassifier = ((u8, u8), (u8, u8), Vec<u8>, Option<HashSet<String>>);
 
 fn get_python_requires_with_classifier(
-    table: &RefMut<Vec<SyntaxElement>>,
+    table: &[SyntaxElement],
     max_supported_python: (u8, u8),
     min_supported_python: (u8, u8),
 ) -> MaxMinPythonWithClassifier {

rust/src/project.rs

@@ -5,7 +5,7 @@ use crate::helpers::table::{collapse_sub_tables, for_entries, reorder_table_keys
 #[allow(clippy::too_many_lines)]
 pub fn fix(tables: &mut Tables) {
     collapse_sub_tables(tables, "tool.ruff");
-    let table_element = tables.get(&String::from("tool.ruff"));
+    let table_element = tables.get("tool.ruff");
     if table_element.is_none() {
         return;
     }