Create a manager for conversations.
Review Request #677 — Created May 28, 2021 and submitted
I Skipped unit tests because need a connection and full protocol implementation to do it properly.
Compiled and ran with an IRC account.
Description | From | Last Updated |
---|---|---|
Do you really need two vars? Seems like g_list_delete_link would work. |
|
|
Should be l->data, I believe. |
|
|
Actually, now it seems like purple_conversations_get_all is called multiple times; can you not free here and start from list again … |
|
|
Extra blank line. |
|
|
Same comments as the previous function. |
|
|
gint? |
|
|
#PurpleConversation\s or something like that? |
|
|
an |
|
|
@name |
|
|
@name |
|
|
All callers have already checked these things, so they're redundant, except for purple_conversation_manager_find_chat_by_id, which passes name=NULL and is broken by … |
|
|
!func(conversation, userdata) |
|
|
The hash table should be freed here? |
|
|
Don't need the if. |
|
|
Funny wrap indent. |
|
|
Funny wrap indent. |
|
|
Why is this guint instead of gint? |
|
|
Is gtkblist->menutrayicon not just created in the below deleted code, and thus unused now? |
|
|
Same questions as in finch. |
|
- Description:
-
~ This will eventually replace the purple_conversations API. I Skipped unit
~ I Skipped unit tests because need a connection and full protocol implementation to do it properly.
- tests because need a connection and full protocol implementation to do it - properly. - Commit:
-
3b164a92379c2df720f9bd1e
-
-
-
-
Actually, now it seems like
purple_conversations_get_all
is called multiple times; can you not free here and start fromlist
again later? I assume the conversation count doesn't change. -
-
-
-
-
-
-
-
All callers have already checked these things, so they're redundant, except for
purple_conversation_manager_find_chat_by_id
, which passesname=NULL
and is broken by thisname != NULL
check. -
-
-
-
-
-
-
-