Skip to content

Commit

Permalink
Cleanup code
Browse files Browse the repository at this point in the history
* Remove previously commented out code
* Remove 'use Data::Dumper'

Signed-off-by: Mikko Johannes Koivunalho <[email protected]>
  • Loading branch information
mikkoi committed May 12, 2022
1 parent c99b4b4 commit 69af734
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 25 deletions.
2 changes: 0 additions & 2 deletions lib/Dancer2/ConfigReader.pm
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@ has environment => (
has config_readers => (
is => 'ro',
lazy => 1,
#isa => ArrayRef[ InstanceOf['Dancer2::Core::Role::ConfigReader'] ],
isa => ArrayRef,
builder => '_build_config_readers',
);
Expand All @@ -73,7 +72,6 @@ sub _build_config {
my ($self) = @_;

my $default = $self->default_config;
use Data::Dumper;
my $config = Hash::Merge::Simple->merge(
$default,
map {
Expand Down
11 changes: 0 additions & 11 deletions lib/Dancer2/Core/App.pm
Original file line number Diff line number Diff line change
Expand Up @@ -445,17 +445,6 @@ sub _build_config {
return $config;
}

# around _build_config => sub {
# my ( $orig, $self ) = @_;
# my $config = $self->$orig;
#
# if ( $config && $config->{'engines'} ) {
# $self->_validate_engine($_) for keys %{ $config->{'engines'} };
# }
#
# return $config;
# };

sub _build_response {
my $self = shift;
return Dancer2::Core::Response->new(
Expand Down
12 changes: 0 additions & 12 deletions lib/Dancer2/Core/Role/HasConfig.pm
Original file line number Diff line number Diff line change
Expand Up @@ -77,18 +77,6 @@ sub _set_config_entry {
$self->config->{$name} = $value;
}

# Who calls this method?
# sub _compile_config {
# my ( $self, $config ) = @_;
#
# foreach my $key ( keys %{$config} ) {
# my $value = $config->{$key};
# $config->{$key} =
# $self->_compile_config_entry( $key, $value, $config );
# }
# return $config;
# }

sub settings { shift->config }

sub setting {
Expand Down

0 comments on commit 69af734

Please sign in to comment.