Create the new PurpleContactManager

Review Request #1850 — Created Sept. 28, 2022 and submitted

Information

pidgin/pidgin
default

Reviewers

Create the new PurpleContactManager

Ran the unit tests

Summary ID
Create the new PurpleContactManager
a27ad8b2ee3b4fc26ade164a10c9e5569a2d2163
Description From Last Updated

1) "will be return" -> "will be returned" 2) "will returned" -> "will be returned"

ivanhoeivanhoe

"coniact" -> "contact"

ivanhoeivanhoe

Missing a word after the, I assume contact

lifesfadedlifesfaded

This is not freed properly it seems. ==26145== 167 (88 direct, 79 indirect) bytes in 1 blocks are definitely lost …

ivanhoeivanhoe

This seems to be leaking memory (both contact1 and contact2) ==26145== 163 (88 direct, 75 indirect) bytes in 1 blocks …

ivanhoeivanhoe

contact1 and contact2 seem not to be freed properly, similiar to the issue above.

ivanhoeivanhoe
ivanhoe
  1. 
      
  2. libpurple/purplecontactmanager.h (Diff revision 1)
     
     

    1) "will be return" -> "will be returned"
    2) "will returned" -> "will be returned"

  3. libpurple/purpleprivate.h (Diff revision 1)
     
     

    "coniact" -> "contact"

  4. libpurple/tests/test_contact_manager.c (Diff revision 1)
     
     

    This is not freed properly it seems.

    ==26145== 167 (88 direct, 79 indirect) bytes in 1 blocks are definitely lost in loss record 6,822 of 7,309
    ==26145==    at 0x48447E5: malloc (vg_replace_malloc.c:381)
    ==26145==    by 0x4A3DE68: g_malloc (in /usr/lib64/libglib-2.0.so.0.7200.3)
    ==26145==    by 0x4A562C0: g_slice_alloc (in /usr/lib64/libglib-2.0.so.0.7200.3)
    ==26145==    by 0x4A56939: g_slice_alloc0 (in /usr/lib64/libglib-2.0.so.0.7200.3)
    ==26145==    by 0x49BB921: g_type_create_instance (in /usr/lib64/libgobject-2.0.so.0.7200.3)
    ==26145==    by 0x49A0C4C: g_object_new_internal (in /usr/lib64/libgobject-2.0.so.0.7200.3)
    ==26145==    by 0x49A261C: g_object_new_valist (in /usr/lib64/libgobject-2.0.so.0.7200.3)
    ==26145==    by 0x49A2B50: g_object_new (in /usr/lib64/libgobject-2.0.so.0.7200.3)
    ==26145==    by 0x48D34AD: purple_contact_new (purplecontact.c:327)
    ==26145==    by 0x10AE5D: test_purple_contact_manager_remove_all (test_contact_manager.c:172)
    ==26145==    by 0x4A60C05: g_test_run_suite_internal (in /usr/lib64/libglib-2.0.so.0.7200.3)
    ==26145==    by 0x4A6092A: g_test_run_suite_internal (in /usr/lib64/libglib-2.0.so.0.7200.3)
    
  5. libpurple/tests/test_contact_manager.c (Diff revision 1)
     
     

    This seems to be leaking memory (both contact1 and contact2)

    ==26145== 163 (88 direct, 75 indirect) bytes in 1 blocks are definitely lost in loss record 6,820 of 7,309
    ==26145==    at 0x48447E5: malloc (vg_replace_malloc.c:381)
    ==26145==    by 0x4A3DE68: g_malloc (in /usr/lib64/libglib-2.0.so.0.7200.3)
    ==26145==    by 0x4A562C0: g_slice_alloc (in /usr/lib64/libglib-2.0.so.0.7200.3)
    ==26145==    by 0x4A56939: g_slice_alloc0 (in /usr/lib64/libglib-2.0.so.0.7200.3)
    ==26145==    by 0x49BB921: g_type_create_instance (in /usr/lib64/libgobject-2.0.so.0.7200.3)
    ==26145==    by 0x49A0C4C: g_object_new_internal (in /usr/lib64/libgobject-2.0.so.0.7200.3)
    ==26145==    by 0x49A261C: g_object_new_valist (in /usr/lib64/libgobject-2.0.so.0.7200.3)
    ==26145==    by 0x49A2B50: g_object_new (in /usr/lib64/libgobject-2.0.so.0.7200.3)
    ==26145==    by 0x48D34AD: purple_contact_new (purplecontact.c:327)
    ==26145==    by 0x10B115: test_purple_contact_manager_find_with_username (test_contact_manager.c:201)
    ==26145==    by 0x4A60C05: g_test_run_suite_internal (in /usr/lib64/libglib-2.0.so.0.7200.3)
    ==26145==    by 0x4A6092A: g_test_run_suite_internal (in /usr/lib64/libglib-2.0.so.0.7200.3)
    
  6. libpurple/tests/test_contact_manager.c (Diff revision 1)
     
     

    contact1 and contact2 seem not to be freed properly, similiar to the issue above.

  7. 
      
lifesfaded
  1. 
      
  2. libpurple/tests/test_contact_manager.c (Diff revision 1)
     
     

    Missing a word after the, I assume contact

  3. 
      
grim
ivanhoe
  1. Ship It!
  2. 
      
grim
Review request changed

Status: Closed (submitted)

Loading...