From 9cf4fc76d0446ae4133e70f5a0e6b5a980e4b94b Mon Sep 17 00:00:00 2001
From: David Anderson <davea@ssl.berkeley.edu>
Date: Mon, 14 Nov 2011 23:45:08 +0000
Subject: [PATCH] - client: improve messages for the above

svn path=/trunk/boinc/; revision=24594
---
 checkin_notes        |  6 ++++++
 client/log_flags.cpp | 22 +++++++++++++---------
 2 files changed, 19 insertions(+), 9 deletions(-)

diff --git a/checkin_notes b/checkin_notes
index b387c03db5..e0da49d484 100644
--- a/checkin_notes
+++ b/checkin_notes
@@ -8456,3 +8456,9 @@ David  14 Nov 2011
         cc_config.cpp,h
     client/
         log_flags.cpp
+
+David  14 Nov 2011
+    - client: improve messages for the above
+
+    client/
+        log_flags.cpp
diff --git a/client/log_flags.cpp b/client/log_flags.cpp
index 08bec9c332..e5c12e40b0 100644
--- a/client/log_flags.cpp
+++ b/client/log_flags.cpp
@@ -141,8 +141,12 @@ static void show_exclude_gpu(EXCLUDE_GPU& e) {
             for (unsigned int i=0; i<gstate.apps.size(); i++) {
                 app = gstate.apps[i];
                 if (app->project != p) continue;
+				if (!app_list.empty()) {
+					app_list += ", ";
+				}
+                app_list += "'";
                 app_list += app->name;
-                app_list += " ";
+                app_list += "'";
             }
             msg_printf(p, MSG_USER_ALERT,
                 "A GPU exclusion in your cc_config.xml file specifies a non-existent application '%s'.  Existing applications: %s",
@@ -276,14 +280,14 @@ int CONFIG::parse_options_client(XML_PARSER& xp) {
             COPROC c;
             retval = c.parse(xp);
             if (retval) {
-                msg_printf(0, MSG_USER_ALERT,
-                    "Can't parse <coproc> in cc_config.xml"
+                msg_printf_notice(NULL, false, NULL,
+                    "Can't parse <coproc> element in cc_config.xml"
                 );
             }
             retval = coprocs.add(c);
             if (retval) {
-                msg_printf(0, MSG_USER_ALERT,
-                    "Duplicate <coproc> in cc_config.xml"
+                msg_printf_notice(NULL, false, NULL,
+                    "Duplicate <coproc> element in cc_config.xml"
                 );
             }
             continue;
@@ -302,8 +306,8 @@ int CONFIG::parse_options_client(XML_PARSER& xp) {
             EXCLUDE_GPU eg;
             retval = eg.parse(xp);
             if (retval) {
-                msg_printf(0, MSG_USER_ALERT,
-                    "Can't parse <exclude_gpu> in cc_config.xml"
+                msg_printf_notice(NULL, false, NULL,
+                    "Can't parse <exclude_gpu> element in cc_config.xml"
                 );
             } else {
                 exclude_gpus.push_back(eg);
@@ -362,8 +366,8 @@ int CONFIG::parse_options_client(XML_PARSER& xp) {
         if (xp.match_tag("proxy_info")) {
             retval = config_proxy_info.parse_config(xp);
             if (retval) {
-                msg_printf(NULL, MSG_USER_ALERT,
-                    "Can't parse <proxy_info> in cc_config.xml"
+                msg_printf_notice(NULL, false, NULL,
+                    "Can't parse <proxy_info> element in cc_config.xml"
                 );
             }
             continue;