Move the GUI frontend doc out of the Qt implementation
[lilass.git] / zenity_dialogue.py
index 2d523fec65f8ba1fafd6168ccca0b74c9601b699..69365e8325d8c8882f06450ca3f28ec5e0f1b6ee 100644 (file)
@@ -12,7 +12,7 @@
 # GNU General Public License for more details.
 #
 # You should have received a copy of the GNU General Public License
-# along with this program (gpl.txt); if not, write to the Free Software
+# along with this program; if not, write to the Free Software
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 
 import subprocess
@@ -22,7 +22,7 @@ def userChoose (title, choices, returns, fallback):
        assert len(choices) == len(returns)
        p = subprocess.Popen(["zenity", "--list", "--text="+title, "--column="]+choices, stdout=subprocess.PIPE)
        switch = dict (zip (choices,returns))
-       for line in p.stdout:
+       for line in p.stdout: # FIXME use p.communicate()[0] instead to get entire stdout and ensure the process terminates. also check p.returncode.
                return switch.get(line.strip(), fallback)
        return fallback