-
-
-
-
-
-
-
-
libpurple/purplechatconversation.c (Diff revision 1) Why not set
priv->updated
directly like inpurple_chat_conversation_ignore
? -
-
libpurple/purplechatconversation.c (Diff revision 1) I don't understand why
users
andflags
needed an extra iteration variable (ul
andfl
) whileextra_msgs
didn't. -
-
Modernizing PurpleChatConversation.
Review Request #583 — Created March 26, 2021 and submitted
I wasn't able to do everything as PurpleConversation still isn't declared with G_DECLARE_DERIVABLE_TYPE.
Compiled ran locally with both xmpp mucs and irc channels.
Description | From | Last Updated |
---|---|---|
PurpleChatUser |
QuLogic | |
@user or @message |
QuLogic | |
Can drop this comment now, I think. |
QuLogic | |
its |
QuLogic | |
Maybe this should be PROP_CHAT_ID? |
QuLogic | |
This seems new; is it really needed right now? |
QuLogic | |
Why not set priv->updated directly like in purple_chat_conversation_ignore? |
QuLogic | |
You deleted the setting of the new values? |
QuLogic | |
I don't understand why users and flags needed an extra iteration variable (ul and fl) while extra_msgs didn't. |
QuLogic | |
Can remove the extra space. |
QuLogic | |
gchar? |
QuLogic | |
Missed that this should be 'referred' earlier. |
QuLogic |
Change Summary:
Address review findings.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+633 -651) |
Change Summary:
rebased and address issues.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+635 -651) |