-
-
-
libpurple/protocols/facebook/facebook.c (Diff revision 1) Is this a user count that can be passed to
_set_user_count
, or a list of usernames? -
-
libpurple/protocols/irc/msgs.c (Diff revision 1) It doesn't look like you're setting the
channel
field? That is used by theirc_chat_join
vfunc. -
-
libpurple/protocols/null/nullprpl.c (Diff revision 1) Don't you need the
room
field for thejoin_chat
vfunc to work? -
-
-
libpurple/purpleroomlistroom.c (Diff revision 1) field
is going to leak since there is no key destroy function, but it seems like everywhere else uses a static key, which makes this annoying.Maybe add a comment for later if this is going to be rewritten differently?
-
-
-
libpurple/roomlist.c (Diff revision 1) Since
purple_roomlist_room_get_components
istransfer none
, this modifies the hash table on the room object directly. Should there be a copy? -
-
pidgin/gtkroomlist.c (Diff revision 1) Doing this after
do_join_cb
would make more sense, and maybe not need the comment? -
-
-
pidgin/resources/Roomlist/roomlist.ui (Diff revision 1) Do you need this model, since it's set at runtime? (Or it doesn't need to be set at runtime?)
Rework the way roomlists work so we can more easily port them to GTK4
Review Request #1293 — Created Feb. 8, 2022 and submitted
Rework the way roomlists work so we can more easily port them to GTK4
Joined rooms on XMPP via the buttons, double clicking, and the context menu.
Summary | ID |
---|---|
85cd7833df73e33edfedff861455ea4e4941cce1 |
Description | From | Last Updated |
---|---|---|
To be done now, or ...? |
QuLogic | |
Is this a user count that can be passed to _set_user_count, or a list of usernames? |
QuLogic | |
Can this be passed to _set_user_count? |
QuLogic | |
It doesn't look like you're setting the channel field? That is used by the irc_chat_join vfunc. |
QuLogic | |
Seems like name was previously the description? |
QuLogic | |
Don't you need the room field for the join_chat vfunc to work? |
QuLogic | |
Same here as irc about the channel field. |
QuLogic | |
Should be spaces? |
QuLogic | |
field is going to leak since there is no key destroy function, but it seems like everywhere else uses a … |
QuLogic | |
whose |
QuLogic | |
This is gone, too. |
QuLogic | |
Since purple_roomlist_room_get_components is transfer none, this modifies the hash table on the room object directly. Should there be a copy? |
QuLogic | |
Weird indent. |
QuLogic | |
Doing this after do_join_cb would make more sense, and maybe not need the comment? |
QuLogic | |
These both seem to be unused? |
QuLogic | |
Seems unrelated, but okay. |
QuLogic | |
Do you need this model, since it's set at runtime? (Or it doesn't need to be set at runtime?) |
QuLogic |
Change Summary:
fix issues, also finch room list is broken on default, so couldn't really test.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+1426 -2082) |
Change Summary:
undo the pidgincolor drive by
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+1420 -2080) |
Change Summary:
rebased
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 4 (+1420 -2080) |