Change Summary:
Fix typos.
Description: |
|
---|
Review Request #2054 — Created Nov. 9, 2022 and updated
Information | |
---|---|
grim | |
pidgin/pidgin | |
default | |
PIDGIN-17687 | |
Reviewers | |
pidgin | |
This is saved in a SQLite database inside the user config directory named
contacts.db
. It does its best to synchronize withPurpleBuddy
, but not
everything emits a signal when it changes.Also, stuff like custom avatars are ignored as those are currently store
on the Contact and we don't have a mapping toPurplePerson
yet.
Ran with the demo account and verified everything about the contacts was stored via the sqlite3 command line.
Summary | |
---|---|
Description | From | Last Updated |
---|---|---|
Figure out how to handle unit tests. This doens't know when we're testing or not, so it always tries to … |
|
|
Does this need a rebase for PurpleContactInfo stuff? |
|
|
Why is this in the account manager header? |
|
|
NULL works here. |
|
|
name and value are not freed here. |
|
|
name and value are not freed at the end of the loop. |
|
|
Not sure why this calls sync_tags but not update_contact_tags? And should everything get a transaction around it? |
|
|
Ditto. |
|
|
Don't need the semicolon, I think? |
|
|
Extra tab? |
|
Fix typos.
Description: |
|
---|
rebase, just found a bug in the unit tests that needs to be addressed yet.
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+974 -10) |
libpurple/purplecontactmanager.c (Diff revision 2) |
---|
name
andvalue
are not freed at the end of the loop.
libpurple/purplecontactmanager.c (Diff revision 2) |
---|
Not sure why this calls
sync_tags
but notupdate_contact_tags
? And should everything get a transaction around it?
rebase and address issues, the unit test failures remain however.
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+994 -10) |
rebase, fix whitespace
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 4 (+994 -10) |