<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 30, 2014 at 10:49 PM, Jonathon Jongsma <span dir="ltr"><<a href="mailto:jjongsma@redhat.com" target="_blank">jjongsma@redhat.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>---<br>
 src/virt-viewer-app.c | 14 ++++++++++++++<br>
 1 file changed, 14 insertions(+)</div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>

<br>
diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c<br>
</div>index 6bd49f4..158a638 100644<br>
<div>--- a/src/virt-viewer-app.c<br>
+++ b/src/virt-viewer-app.c<br>
@@ -242,6 +242,20 @@ virt_viewer_app_save_config(VirtViewerApp *self)<br>
         g_warning("failed to create config directory");<br>
     g_free(dir);<br>
<br>
+    if (priv->uuid && priv->guest_name) {<br>
+        // if there's no comment for this uuid settings group, add a comment<br>
+        // with the vm name so user can make sense of it later.<br>
+        gchar* comment = g_key_file_get_comment(priv->config, priv->uuid, NULL, &error);<br></div></blockquote><div><br></div><div>Coding style here, use: gchar *comment (...)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
+        if (error) {<br>
+            g_warning("Unable to get comment from key file: %s", error->message);<br>
+            g_clear_error(&error);<br>
+        } else {<br>
</div>+            if (!comment || *comment == '\0')<br>
<div>+                g_key_file_set_comment(priv->config, priv->uuid, NULL, priv->guest_name, NULL);<br>
+        }<br>
</div><div><div>+        g_free(comment);<br>
+    }<br>
+<br>
     if ((data = g_key_file_to_data(priv->config, NULL, &error)) == NULL ||<br>
         !g_file_set_contents(priv->config_file, data, -1, &error)) {<br>
         g_warning("Couldn't save configuration: %s", error->message);<br>
--<br>
1.9.3<br>
<br>
_______________________________________________<br>
virt-tools-list mailing list<br>
<a href="mailto:virt-tools-list@redhat.com" target="_blank">virt-tools-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/virt-tools-list" target="_blank">https://www.redhat.com/mailman/listinfo/virt-tools-list</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Fabiano Fidêncio</div>
</div></div>