Merge pull request #168 from yast/optimize_installation

Optimize installation
yast · Jun 28, 2016 · a73638b · a73638b
2 parents 60ae5b4 + 6f57cca
commit a73638b
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 7 deletions.
diff --git a/package/yast2-packager.changes b/package/yast2-packager.changes
@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Tue Jun 28 14:15:15 UTC 2016 - jreidinger@suse.com
+
+- optimize slide show size computing to improve installation
+  performance (bnc#986649)
+- 3.1.105
+
 -------------------------------------------------------------------
 Mon Jun 20 08:04:22 UTC 2016 - lslezak@suse.cz
 

diff --git a/package/yast2-packager.spec b/package/yast2-packager.spec
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-packager
-Version:        3.1.104
+Version:        3.1.105
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

diff --git a/src/modules/PackageSlideShow.rb b/src/modules/PackageSlideShow.rb
@@ -129,12 +129,10 @@ def GetPackageSummary
     # Sum up all list items
     #
     def ListSum(sizes)
-      sizes = deep_copy(sizes)
-      sum = 0
-
-      Builtins.foreach(sizes) { |item| sum = Ops.add(sum, item) if item != -1 }
-
-      sum
+      sizes.each_with_object(0) do |i, r|
+        next if i == -1
+        r += i
+      end
     end
 
     # Sum up all positive list items, but cut off individual items at a maximum value.