Improve decision regarding trimming the root

When the user doesn't want root trimming, we won't trim even
when a search is active.

Fix #434

New decision process:

1. we never want to trim the root if there's a sort
2. if the user don't want root trimming, we don't trim
3. if there's a pattern, we try to show at least root matches
4. in other cases, as the user wants trimming, we trim

1 files changed, 10 insertions, 2 deletions

diff --git a/src/tree_build/builder.rs b/src/tree_build/builder.rs
index f5e8752..37e406b 100644
--- a/src/tree_build/builder.rs
+++ b/src/tree_build/builder.rs
@@ -87,8 +87,16 @@ impl<'c> TreeBuilder<'c> {
             None
         };
         let root_id = BLine::from_root(&mut blines, path, root_ignore_chain, &options)?;
-        let trim_root = options.pattern.is_some()
-            || (options.trim_root && !options.sort.is_some());
+        let trim_root = match (options.trim_root, options.pattern.is_some(), options.sort.is_some()) {
+            // we never want to trim the root if there's a sort
+            (_, _, true) => false,
+            // if the user don't want root trimming, we don't trim
+            (false, _, _) => false,
+            // if there's a pattern, we try to show at least root matches
+            (_, true, _) => false,
+            // in other cases, as the user wants trimming, we trim
+            _ => true,
+        };
         Ok(TreeBuilder {
             options,
             targeted_size,