From 5a95a617420743900ebb6e0fd168eee08c60af28 Mon Sep 17 00:00:00 2001
From: Arvin Schnell <aschnell@suse.de>
Date: Fri, 2 Aug 2013 10:10:05 +0200
Subject: [PATCH] - avoid passing by reference for MakeNavigationTree

---
 src/include/partitioning/ep-main.rb | 60 ++++++++++++++---------------
 1 file changed, 28 insertions(+), 32 deletions(-)

diff --git a/src/include/partitioning/ep-main.rb b/src/include/partitioning/ep-main.rb
index 4b5b86807..57c7d72a1 100644
--- a/src/include/partitioning/ep-main.rb
+++ b/src/include/partitioning/ep-main.rb
@@ -108,11 +108,11 @@ def UpdateMainStatus
     end
 
 
-    def MakeNavigationTree(open_items, tree, data)
+    def MakeNavigationTree(open_items)
       open_items = deep_copy(open_items)
       # TODO: somehow use AlwaysHideDisk
 
-      data.value = {
+      data = {
         :all         => {
           :create => fun_ref(method(:CreateAllPanel), "void (any)"),
           :handle => fun_ref(method(:HandleAllPanel), "void (any, map)")
@@ -211,8 +211,8 @@ def MakeNavigationTree(open_items, tree, data)
             tmp,
             Item(Id(part_device), part_displayname, open.call(part_device))
           )
-          data.value = Builtins.add(
-            data.value,
+          data = Builtins.add(
+            data,
             part_device,
             Builtins.union(a, { :user_data => part_device })
           )
@@ -233,8 +233,8 @@ def MakeNavigationTree(open_items, tree, data)
               )
             )
           )
-          data.value = Builtins.add(
-            data.value,
+          data = Builtins.add(
+            data,
             disk_device,
             Builtins.union(b, { :user_data => disk_device })
           )
@@ -368,7 +368,7 @@ def MakeNavigationTree(open_items, tree, data)
       short_hostname = Hostname.CurrentHostname
 
       # TODO: same ordering as with IterateTargetMap
-      tree.value = [
+      tree = [
         Item(
           Id(:all),
           term(:icon, StorageIcons.all_icon),
@@ -449,8 +449,8 @@ def MakeNavigationTree(open_items, tree, data)
 
       if UI.HasSpecialWidget(:Graph)
         # tree node label
-        tree.value = Builtins.add(
-          tree.value,
+        tree = Builtins.add(
+          tree,
           Item(
             Id(:devicegraph),
             term(:icon, StorageIcons.graph_icon),
@@ -459,8 +459,8 @@ def MakeNavigationTree(open_items, tree, data)
           )
         )
         # tree node label
-        tree.value = Builtins.add(
-          tree.value,
+        tree = Builtins.add(
+          tree,
           Item(
             Id(:mountgraph),
             term(:icon, StorageIcons.graph_icon),
@@ -472,8 +472,8 @@ def MakeNavigationTree(open_items, tree, data)
 
 
       # tree node label
-      tree.value = Builtins.add(
-        tree.value,
+      tree = Builtins.add(
+        tree,
         Item(
           Id(:summary),
           term(:icon, StorageIcons.summary_icon),
@@ -483,8 +483,8 @@ def MakeNavigationTree(open_items, tree, data)
       )
 
       # tree node label
-      tree.value = Builtins.add(
-        tree.value,
+      tree = Builtins.add(
+        tree,
         Item(
           Id(:settings),
           term(:icon, StorageIcons.settings_icon),
@@ -495,8 +495,8 @@ def MakeNavigationTree(open_items, tree, data)
 
       if Mode.normal
         # tree node label
-        tree.value = Builtins.add(
-          tree.value,
+        tree = Builtins.add(
+          tree,
           Item(
             Id(:log),
             term(:icon, StorageIcons.log_icon),
@@ -506,22 +506,24 @@ def MakeNavigationTree(open_items, tree, data)
         )
       end
 
-      nil
+      return tree, data
+
     end
+
+
     def UpdateNavigationTree(new_focus)
       new_focus = deep_copy(new_focus)
-      tree = []
-      data = {}
+
       open_items = Convert.to_map(UI.QueryWidget(Id(:tree), :OpenItems))
-      tree_ref = arg_ref(tree)
-      data_ref = arg_ref(data)
-      MakeNavigationTree(open_items, tree_ref, data_ref)
-      tree = tree_ref.value
-      data = data_ref.value
+
+      tree, data = MakeNavigationTree(open_items)
+
       TreePanel.Update(data, tree, new_focus)
 
       nil
     end
+
+
     def EpContextMenuDevice(device)
       target_map = Storage.GetTargetMap
 
@@ -659,13 +661,7 @@ def ExpertPartitioner
 
       Storage.CreateTargetBackup("expert-partitioner")
 
-      tree = []
-      data = {}
-      tree_ref = arg_ref(tree)
-      data_ref = arg_ref(data)
-      MakeNavigationTree({ :all => "ID" }, tree_ref, data_ref)
-      tree = tree_ref.value
-      data = data_ref.value
+      tree, data = MakeNavigationTree({ :all => "ID" })
 
       back_label = Label.BackButton
       next_label = Mode.normal ? Label.FinishButton : Label.AcceptButton