From d6cbb904419bb5e5c0efcf2bf6cef40f52be94e6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20=27Necoro=27=20Neumann?= <necoro@necoro.net>
Date: Mon, 9 Feb 2009 17:49:16 +0100
Subject: Change 'split(" ")' to single 'split()' to also catch multiple spaces
 and tabs

---
 portato/backend/flags.py    | 4 ++--
 portato/gui/windows/main.py | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/portato/backend/flags.py b/portato/backend/flags.py
index 2b6b266..c80bf8d 100644
--- a/portato/backend/flags.py
+++ b/portato/backend/flags.py
@@ -381,7 +381,7 @@ def write_use_flags ():
                         while i < line: # stop at the given line
                             lines.append(f.readline())
                             i += 1
-                        l = f.readline().split(" ")
+                        l = f.readline().split()
                         
                         # delete or insert
                         if delete:
@@ -396,7 +396,7 @@ def write_use_flags ():
                         file_cache[file] = lines
 
                 else: # in cache
-                    l = file_cache[file][line-1].split(" ")
+                    l = file_cache[file][line-1].split()
                     if delete:
                         remove(flag, l)
                     else:
diff --git a/portato/gui/windows/main.py b/portato/gui/windows/main.py
index 0882c2a..6f7423c 100644
--- a/portato/gui/windows/main.py
+++ b/portato/gui/windows/main.py
@@ -162,7 +162,7 @@ class PackageTable:
             self.linkBox.remove(c)
         
         text = pkg.get_package_settings("HOMEPAGE")
-        texts = text.split(" ")
+        texts = text.split()
         ftexts = []
 
         for count, t in enumerate(texts):
-- 
cgit v1.2.3


From 58e5f45f4c1f254e626e144c52ff527fcf8b0b59 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20=27Necoro=27=20Neumann?= <necoro@necoro.net>
Date: Mon, 9 Feb 2009 18:07:19 +0100
Subject: Re-honor the newline add the end :). (It was removed by normal
 'split()'). Other minor code enhancements too

---
 portato/backend/flags.py | 39 ++++++++++++++++++++-------------------
 1 file changed, 20 insertions(+), 19 deletions(-)

diff --git a/portato/backend/flags.py b/portato/backend/flags.py
index c80bf8d..baa1f37 100644
--- a/portato/backend/flags.py
+++ b/portato/backend/flags.py
@@ -343,22 +343,22 @@ def write_use_flags ():
     """This writes our changed useflags into the file."""
     global newUseFlags, useFlags
 
+    def combine (list):
+        """Shortcut for reverting the list into a string."""
+        return " ".join(list)+"\n"
+
     def insert (flag, list):
         """Shortcut for inserting a new flag right after the package-name."""
         list.insert(1,flag)
     
     def remove (flag, list):
         """Removes a flag."""
-        try:
-            list.remove(flag)
-        except ValueError: # flag is given as flag\n
-            list.remove(flag+"\n")
-            list.append("\n") #re-insert the newline
+        list.remove(flag)
 
         # no more flags there - comment it out
-        if len(list) == 1 or list[1][0] in ("#","\n"):
+        if len(list) == 1 or list[1][0] == "#":
             list[0] = "#"+list[0]
-            insert("#removed by portato#",list)
+            list.append("#removed by portato#")
 
     file_cache = {} # cache for having to read the file only once: name->[lines]
     for cpv in newUseFlags:
@@ -388,7 +388,7 @@ def write_use_flags ():
                             remove(flag,l)
                         else:
                             insert(flag,l)
-                        lines.append(" ".join(l))
+                        lines.append(combine(l))
                         
                         # read the rest
                         lines.extend(f.readlines())
@@ -400,29 +400,30 @@ def write_use_flags ():
                     if delete:
                         remove(flag, l)
                     else:
-                        insert(flag,l)
-                    file_cache[file][line-1] = " ".join(l)
+                        insert(flag, l)
+                    file_cache[file][line-1] = combine(l)
 
         if flagsToAdd:
         # write new lines
             msg = "\n#portato update#\n"
+            comb = combine(flagsToAdd)
             if CONFIG["usePerVersion"]: # add on a per-version-base
-                msg += "=%s %s\n" % (cpv, ' '.join(flagsToAdd))
+                msg += "=%s %s" % (cpv, comb)
             else: # add on a per-package-base
                 list = system.split_cpv(cpv)
-                msg += "%s/%s %s\n" % (list[0], list[1], ' '.join(flagsToAdd))
+                msg += "%s/%s %s" % (list[0], list[1], combine)
+
             if not file in file_cache:
-                f = open(file, "a")
-                f.write(msg)
-                f.close()
+                with open(file, "a") as f:
+                    f.write(msg)
             else:
                 file_cache[file].append(msg)
     
     # write to disk
-    for file in file_cache.keys():
-        f = open(file, "w")
-        f.writelines(file_cache[file])
-        f.close()
+    for file in file_cache:
+        with open(file, "w") as f:
+            f.writelines(file_cache[file])
+
     # reset
     useFlags = {}
     newUseFlags = {}
-- 
cgit v1.2.3