Remove -e switch, but refactor load_rcfile to make it easy to use the "do FILE" like...
Sartak [Sat, 22 Sep 2007 01:07:54 +0000 (01:07 +0000)]
git-svn-id: http://dev.catalyst.perl.org/repos/bast/trunk/Devel-REPL@3776 bd8105ee-0ff8-0310-8827-fb3f25b6796d

lib/Devel/REPL/Script.pm

index 6da9202..aab0cc2 100644 (file)
@@ -17,10 +17,6 @@ has 'profile' => (
   is => 'ro', isa => 'Str', required => 1, default => sub { 'Default' },
 );
 
-has 'e' => (
-  is => 'ro', isa => 'ArrayRef', default => sub { [] },
-);
-
 has '_repl' => (
   is => 'ro', isa => 'Devel::REPL', required => 1,
   default => sub { Devel::REPL->new() }
@@ -30,7 +26,6 @@ sub BUILD {
   my ($self) = @_;
   $self->load_profile($self->profile);
   $self->load_rcfile($self->rcfile);
-  $self->load_scripts($self->e);
 }
 
 sub load_profile {
@@ -50,25 +45,30 @@ sub load_rcfile {
     $rc_file = File::Spec->catfile(File::HomeDir->my_home, '.re.pl', $rc_file);
   }
 
-  if (-r $rc_file) {
-    open RCFILE, '<', $rc_file || die "Couldn't open ${rc_file}: $!";
-    my $rc_data;
-    { local $/; $rc_data = <RCFILE>; }
-    close RCFILE; # Don't care if this fails
-    $self->eval_rcdata($rc_data);
-    warn "Error executing rc file ${rc_file}: $@\n" if $@;
-  }
+  $self->apply_script($rc_file);
 }
 
-sub load_scripts {
-  my ($self, $scripts) = @_;
+sub apply_script {
+  my ($self, $script, $warn_on_unreadable) = @_;
 
-  for (@$scripts) {
-    do $_;
+  if (!-e $script) {
+    warn "File '$script' does not exist" if $warn_on_unreadable;
+    return;
   }
+  elsif (!-r _) {
+    warn "File '$script' is unreadable" if $warn_on_unreadable;
+    return;
+  }
+
+  open RCFILE, '<', $script or die "Couldn't open ${script}: $!";
+  my $rc_data;
+  { local $/; $rc_data = <RCFILE>; }
+  close RCFILE; # Don't care if this fails
+  $self->eval_script($rc_data);
+  warn "Error executing script ${script}: $@\n" if $@;
 }
 
-sub eval_rcdata {
+sub eval_script {
   my ($self, $data) = @_;
   local $CURRENT_SCRIPT = $self;
   $self->_repl->eval($data);