changeset 27474:d7ac2ad0db5a

Add a comment about some strange behavior that I noticed
author Mark Doliner <mark@kingant.net>
date Mon, 06 Jul 2009 23:24:37 +0000
parents b9790d3e2a0a
children 57fdb7853fc7
files libpurple/protocols/qq/buddy_info.c
diffstat 1 files changed, 9 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/libpurple/protocols/qq/buddy_info.c	Mon Jul 06 22:21:20 2009 +0000
+++ b/libpurple/protocols/qq/buddy_info.c	Mon Jul 06 23:24:37 2009 +0000
@@ -530,6 +530,15 @@
 	return icon_name;
 }
 
+/*
+ * Hmm, this function is a bit weird.
+ * 1. The pref "/plugins/prpl/qq/icon_dir" doesn't exist
+ * 2. QQ_BUDDY_ICON_DIR is set in Makefile.am, but it's set to a
+ *    directory that doesn't exist
+ * 3. Why would the icon path be different for Windows and Linux?
+ * 4. Why do we use this in the first place?  Setting your icon
+ *    should go through the account editor.
+ */
 gchar *qq_get_icon_path(gchar *icon_name)
 {
 	gchar *icon_path;