From b83c55e97fa95713f66198c42577c284036ecefa Mon Sep 17 00:00:00 2001 From: Florian Pritz Date: Sat, 1 Apr 2017 20:23:16 +0200 Subject: Do not pass $opts to main class The only reason this was done, was for the debug output. This no longer requires large amounts of memory or time (simple count(*) in the db now) so this is no longer necessary. Signed-off-by: Florian Pritz --- lib/App/BorgRestore.pm | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) (limited to 'lib') diff --git a/lib/App/BorgRestore.pm b/lib/App/BorgRestore.pm index abff5e1..7fa3251 100644 --- a/lib/App/BorgRestore.pm +++ b/lib/App/BorgRestore.pm @@ -64,7 +64,6 @@ Florian Pritz Ebluewind@xinu.atE sub new { my $class = shift; - my $opts = shift; my $deps = shift; my $self = {}; @@ -73,7 +72,6 @@ sub new { my $db_path = App::BorgRestore::Settings::get_cache_path('archives.db'); # TODO: make db_path configurable, probably settings too - $self->{opts} = $opts; $self->{borg} = $deps->{borg} // App::BorgRestore::Borg->new(); $self->{db} = $deps->{db} // App::BorgRestore::DB->new($db_path); @@ -82,13 +80,11 @@ sub new { sub new_no_defaults { my $class = shift; - my $opts = shift; my $deps = shift; my $self = {}; bless $self, $class; - $self->{opts} = $opts; $self->{borg} = $deps->{borg}; $self->{db} = $deps->{db}; @@ -352,9 +348,7 @@ sub build_archive_cache { $self->handle_removed_archives($borg_archives); $self->handle_added_archives($borg_archives); - if ($self->{opts}->{debug}) { - $log->debugf("DB contains information for %d archives in %d rows", scalar(@{$self->{db}->get_archive_names()}), $self->{db}->get_archive_row_count()); - } + $log->debugf("DB contains information for %d archives in %d rows", scalar(@{$self->{db}->get_archive_names()}), $self->{db}->get_archive_row_count()); } sub save_node { -- cgit v1.2.3-24-g4f1b