[ARVADOS] updated: 0ecbb08536d02433272e62cf1878edd7927f66e1

git at public.curoverse.com git at public.curoverse.com
Tue Dec 2 16:17:40 EST 2014


Summary of changes:
 sdk/cli/bin/crunch-job | 75 +++++++++++++++++++++-----------------------------
 1 file changed, 31 insertions(+), 44 deletions(-)

       via  0ecbb08536d02433272e62cf1878edd7927f66e1 (commit)
       via  59a100ec439092d24dd918cc4e9ed875b52797b5 (commit)
       via  6cd08f799a2feb762ec8dc031675ea815726d755 (commit)
       via  a06d87af46cb235b9764d9383e46c3e02823c49c (commit)
       via  7f83b8600e45216c2de47cad06b7eaff4ad5c9cd (commit)
       via  dc706977b754f7b5b0e5835ef80a2b59bc1773f8 (commit)
      from  6c36016f708ce91a4287bd2723827d009ba93a1d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 0ecbb08536d02433272e62cf1878edd7927f66e1
Merge: 6c36016 59a100e
Author: Tim Pierce <twp at curoverse.com>
Date:   Tue Dec 2 16:17:26 2014 -0500

    Merge branch '4621-crunch-memory-usage'
    
    Refs #4621.


commit 59a100ec439092d24dd918cc4e9ed875b52797b5
Author: Tim Pierce <twp at curoverse.com>
Date:   Tue Dec 2 16:12:42 2014 -0500

    4621: undo lean_uri_escape changes

diff --git a/sdk/perl/lib/Arvados/Request.pm b/sdk/perl/lib/Arvados/Request.pm
index f2a9b63..07ca763 100644
--- a/sdk/perl/lib/Arvados/Request.pm
+++ b/sdk/perl/lib/Arvados/Request.pm
@@ -49,9 +49,9 @@ sub process_request
     my $content;
     while (($p, $v) = each %content) {
         $content .= '&' unless $content eq '';
-        $content .= lean_uri_escape($p);
+        $content .= uri_escape($p);
         $content .= '=';
-        $content .= lean_uri_escape($v);
+        $content .= uri_escape($v);
     }
     $self->{'req'}->content_type("application/x-www-form-urlencoded; charset='utf8'");
     $self->{'req'}->content(Encode::encode('utf8', $content));
@@ -94,12 +94,4 @@ sub get_headers
     ""
 }
 
-# lean_uri_escape consumes about half as much memory
-# as URI::Escape::uri_escape.
-sub lean_uri_escape {
-    my ($text) = @_;
-    $text =~ s/([^A-Za-z0-9\-\._~])/$URI::Escape::escapes{$&}/ge;
-    return $text;
-}
-
 1;

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list