[ARVADOS] updated: 0e93a99cee72ca54650c6e9aa41af3fb5adc869a

[git at public.curoverse.com]

Summary of changes:
 services/api/app/models/collection.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

       via  0e93a99cee72ca54650c6e9aa41af3fb5adc869a (commit)
      from  d80be38b5be2c607d3aa0855c69b9f4e0c3fbed0 (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 0e93a99cee72ca54650c6e9aa41af3fb5adc869a
Author: radhika <radhika at curoverse.com>
Date:   Mon Jun 8 15:39:37 2015 -0400

    6203: Use manifest.each_line and line.rstrip! instead of manifest.split("\n").
    Performance was comparable in both cases; though each_line itself is twice as fast (we need to do strip, which eats away the gain).

diff --git a/services/api/app/models/collection.rb b/services/api/app/models/collection.rb
index 4fab364..1d11315 100644
--- a/services/api/app/models/collection.rb
+++ b/services/api/app/models/collection.rb
@@ -222,7 +222,8 @@ class Collection < ArvadosModel
 
     new_lines = []
     lines = manifest.split("\n")
-    lines.each do |line|
+    manifest.each_line do |line|
+      line.rstrip!
       words = line.split(' ')
       new_words = []
       words.each do |word|

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


hooks/post-receive
--