refactor: remove unnecesary code

src/alejandra_engine/src/children.rs

@@ -15,10 +15,9 @@ pub enum Trivia {
 }
 
 impl Children {
-    pub fn new_with_configuration(
+    pub fn new(
         build_ctx: &crate::builder::BuildCtx,
         node: &rnix::SyntaxNode,
-        with_newlines: bool,
     ) -> Children {
         let mut children = Vec::new();
 
@@ -46,10 +45,7 @@ impl Children {
                             });
                         }
                         rnix::SyntaxKind::TOKEN_WHITESPACE => {
-                            if with_newlines
+                            if crate::utils::count_newlines(token.text()) > 0 {
-                                && crate::utils::count_newlines(token.text())
-                                    > 0
-                            {
                                 children.push(Child {
                                     element: token.clone().into(),
                                     pos:     pos.clone(),
@@ -72,13 +68,6 @@ impl Children {
         Children { children, current_index: 0 }
     }
 
-    pub fn new(
-        build_ctx: &crate::builder::BuildCtx,
-        node: &rnix::SyntaxNode,
-    ) -> Children {
-        Children::new_with_configuration(build_ctx, node, false)
-    }
-
     pub fn get(&mut self, index: usize) -> Option<Child> {
         if index + 1 > self.children.len() {
             None

src/alejandra_engine/src/parsers/if_else.rs

@@ -18,9 +18,7 @@ pub fn parse(
 ) -> IfElse {
     let mut if_else = IfElse::default();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     // if
     children.get_next().unwrap();

src/alejandra_engine/src/parsers/pattern.rs

@@ -23,9 +23,7 @@ pub fn parse(
 ) -> Pattern {
     let mut pattern = Pattern::default();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     // x @
     let child = children.peek_next().unwrap();

src/alejandra_engine/src/rules/apply.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/assert_and_with.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/attr_set.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let items_count = node
         .children_with_tokens()

src/alejandra_engine/src/rules/bin_op_and_or_default.rs

@@ -12,9 +12,7 @@ pub fn rule_with_configuration(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/dynamic.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/inherit.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/key_value.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/lambda.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/let_in.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let items_count = node
         .children()

src/alejandra_engine/src/rules/list.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let items_count = node
         .children_with_tokens()

src/alejandra_engine/src/rules/paren.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let has_comments_or_newlines =
         children.has_comments() || children.has_newlines();

src/alejandra_engine/src/rules/pat_bind.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/pat_entry.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/root.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()

src/alejandra_engine/src/rules/string_interpol.rs

@@ -4,9 +4,7 @@ pub fn rule(
 ) -> std::collections::LinkedList<crate::builder::Step> {
     let mut steps = std::collections::LinkedList::new();
 
-    let mut children = crate::children::Children::new_with_configuration(
+    let mut children = crate::children::Children::new(build_ctx, node);
-        build_ctx, node, true,
-    );
 
     let vertical = children.has_comments()
         || children.has_newlines()