# HG changeset patch
# User Mark Doliner <mark@kingant.net>
# Date 1246922677 0
# Node ID d7ac2ad0db5a52c17cd1c8d3244b9c07c9d4db27
# Parent  b9790d3e2a0a369ffb8e713bbc35d3dd2c4b675e
Add a comment about some strange behavior that I noticed

diff -r b9790d3e2a0a -r d7ac2ad0db5a libpurple/protocols/qq/buddy_info.c
--- 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;