From 9f5bc01a25d1118b0141f322e23834b048a17be9 Mon Sep 17 00:00:00 2001
From: Charlie Fenton <charlief@example.com>
Date: Wed, 4 May 2011 11:14:23 +0000
Subject: [PATCH] MGR: Continue adding Exclusive Apps pane to Computing Prefs
 dialog

svn path=/trunk/boinc/; revision=23495
---
 clientgui/DlgAdvPreferences.cpp | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/clientgui/DlgAdvPreferences.cpp b/clientgui/DlgAdvPreferences.cpp
index 20f5014dee..3ae087079a 100644
--- a/clientgui/DlgAdvPreferences.cpp
+++ b/clientgui/DlgAdvPreferences.cpp
@@ -836,7 +836,12 @@ void CDlgAdvPreferences::OnAddExclusiveApp(wxCommandEvent&) {
         // Strip off path if present
         appNames.Add(theAppName.AfterLast(path_separator));
         
-        // We can only perform minimal validation on remote hosts
+    }
+        
+    for (i=0; i<appNames.Count(); ++i) {
+        // wxFileName::IsFileExecutable() doesn't seem to work on Windows, 
+        // and we can only perform minimal validation on remote hosts, so 
+        // check filename extension on Mac and Win
         if (hostIsMac || hostIsWin) {
             if (!appNames[0].EndsWith(extension)) {
                 errmsg.Printf(_("Application names must end with '%s'"), extension);
@@ -845,9 +850,7 @@ void CDlgAdvPreferences::OnAddExclusiveApp(wxCommandEvent&) {
                 return;
             }
         }
-    }
-        
-    for (i=0; i<appNames.Count(); ++i) {
+
         if (hostIsMac) {
             int suffix = appNames[i].Find('.', true);
             if (suffix != wxNOT_FOUND) {