on second thought, make all empty specs return a lexical sort
groditi [Sat, 11 Oct 2008 20:44:51 +0000 (20:44 +0000)]
lib/Reaction/UI/ViewPort.pm

index d39a108..34c2fee 100644 (file)
@@ -121,15 +121,12 @@ sub event_id_for {
 
 sub sort_by_spec {
   my ($self, $spec, $items) = @_;
-  return [@$items] if not defined $spec;
+  return [sort @$items] unless $spec;
 
   my @order;
   if (ref $spec eq 'ARRAY') {
-    return [sort @$items] unless @$spec;
     @order = @$spec;
-  }
-  elsif (not ref $spec) {
-    return [@$items] unless length $spec;
+  } elsif (not ref $spec) {
     @order = split /\s+/, $spec;
   }