diff options
author | Andrew Gallant <jamslam@gmail.com> | 2018-07-22 10:57:33 -0400 |
---|---|---|
committer | Andrew Gallant <jamslam@gmail.com> | 2018-07-22 10:59:09 -0400 |
commit | 03af61fc7bb384cb594359fee806912ee5e57a98 (patch) | |
tree | 725836b193877d35c20c16e9d1920aa813750950 /src | |
parent | 560dffd2476f16b4b296aae6aca7aec0f0c6ba1c (diff) |
ripgrep: don't skip tar archives
This removes logic from the decompressor for skipping tar archives. This
logic was originally added under the assumption that we probably want to
avoid the cost of reading them. However, this is generally inconsistent
with how ripgrep treats files like tar archives: it should search them
and do binary detection like normal.
Fixes #918
Diffstat (limited to 'src')
-rw-r--r-- | src/decompressor.rs | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/decompressor.rs b/src/decompressor.rs index 235849b7..d25c2f56 100644 --- a/src/decompressor.rs +++ b/src/decompressor.rs @@ -92,10 +92,6 @@ impl DecompressionReader { /// If there is any error in spawning the decompression command, then /// return `None`, after outputting any necessary debug or error messages. pub fn from_path(path: &Path) -> Option<DecompressionReader> { - if is_tar_archive(path) { - debug!("{}: skipping tar archive", path.display()); - return None; - } let extension = match path.extension().and_then(OsStr::to_str) { Some(extension) => extension, None => { |