changeset 13827:6c3c8d0d389f

[gaim-migrate @ 16271] Some comment changes committer: Tailor Script <tailor@pidgin.im>
author Mark Doliner <mark@kingant.net>
date Sat, 17 Jun 2006 01:33:09 +0000
parents 475aed3aacfc
children 71a6a6b22149
files src/protocols/msn/userlist.c src/protocols/oscar/oscar.c src/protocols/yahoo/yahoo_packet.c
diffstat 3 files changed, 13 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/protocols/msn/userlist.c	Fri Jun 16 16:17:20 2006 +0000
+++ b/src/protocols/msn/userlist.c	Sat Jun 17 01:33:09 2006 +0000
@@ -292,6 +292,11 @@
  
 		if (!(user->list_op & (MSN_LIST_AL_OP | MSN_LIST_BL_OP)))
 		{
+			/*
+			 * TODO: The friendly name was NULL for me when I
+			 *       looked at this.  Maybe we should use the store
+			 *       name instead? --KingAnt
+			 */
 			got_new_entry(gc, passport, friendly);
 		}
 	}
@@ -413,7 +418,12 @@
 	if (list_op & MSN_LIST_RL_OP)
 	{
 		/* These are users who have us on their buddy list. */
-		/* TODO: what does store name is when this happens? */
+		/*
+		 * TODO: What is store name set to when this happens?
+		 *       For one of my accounts "something@hotmail.com"
+		 *       the store name was "something."  Maybe we
+		 *       should use the friendly name, instead? --KingAnt
+		 */
 
 		if (!(list_op & (MSN_LIST_AL_OP | MSN_LIST_BL_OP)))
 		{
--- a/src/protocols/oscar/oscar.c	Fri Jun 16 16:17:20 2006 +0000
+++ b/src/protocols/oscar/oscar.c	Sat Jun 17 01:33:09 2006 +0000
@@ -1903,6 +1903,7 @@
 			file = g_fopen(iconfile, "rb");
 			if (file) {
 				/* XXX - Use g_file_get_contents() */
+				/* g_file_get_contents(iconfile, &data, &len, NULL); */
 				int len = fread(buf, 1, st.st_size, file);
 				gaim_debug_info("oscar",
 						   "Sending buddy icon to %s (%d bytes, "
--- a/src/protocols/yahoo/yahoo_packet.c	Fri Jun 16 16:17:20 2006 +0000
+++ b/src/protocols/yahoo/yahoo_packet.c	Sat Jun 17 01:33:09 2006 +0000
@@ -116,7 +116,7 @@
 
 		struct yahoo_pair *pair = g_new0(struct yahoo_pair, 1);
 
-		/* this is weird, and in one of the chat packets, and causes us
+		/* this is weird, and in one of the chat packets, and causes us to
 		 * think all the values are keys and all the keys are values after
 		 * this point if we don't handle it */
 		if (data[pos] == '\0') {