You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by da...@apache.org on 2014/02/12 09:00:30 UTC

svn commit: r1567552 - /subversion/trunk/tools/dist/backport.pl

Author: danielsh
Date: Wed Feb 12 08:00:30 2014
New Revision: 1567552

URL: http://svn.apache.org/r1567552
Log:
backport.pl: code simplification, after r1567519.

* tools/dist/backport.pl
  (parse_entry): Simplify loop body by taking advantages of new loop condition.

Modified:
    subversion/trunk/tools/dist/backport.pl

Modified: subversion/trunk/tools/dist/backport.pl
URL: http://svn.apache.org/viewvc/subversion/trunk/tools/dist/backport.pl?rev=1567552&r1=1567551&r2=1567552&view=diff
==============================================================================
--- subversion/trunk/tools/dist/backport.pl (original)
+++ subversion/trunk/tools/dist/backport.pl Wed Feb 12 08:00:30 2014
@@ -358,12 +358,8 @@ sub parse_entry {
     and shift
     if $_[0] =~ /^(\S*) branch$/ or $_[0] =~ m#branches/(\S+)#;
   while ($_[0] =~ /^(?:r?\d+[,; ]*)+$/) {
-    my $sawrevnum = 0;
-    while ($_[0] =~ s/^r?(\d+)(?:$|[,; ]+)//) {
-      push @revisions, $1;
-      $sawrevnum++;
-    }
-    $sawrevnum ? shift : last;
+    push @revisions, ($_[0] =~ /(\d+)/g);
+    shift;
   }
 
   # summary