Changeset 7528 for trunk/libtransmission/peer-msgs.c

Timestamp:

Dec 29, 2008, 6:10:07 PM (12 years ago)

Author:

charles

Message:

(trunk libT) avoid some unnecessary memory fragmentation... for composited objects that have a tr_publisher, contain the it directly rather than a pointer to one allocated elsewhere on the heap.

File:

• trunk/libtransmission/peer-msgs.c (modified) (6 diffs)

trunk/libtransmission/peer-msgs.c

@@ -297 +297 @@
     tr_torrent *           torrent;
 
-    tr_publisher_t *       publisher;
+    tr_publisher           publisher;
 
     struct evbuffer *      outMessages; /* all the non-piece messages */
@@ -531 +531 @@
     assert( msgs->peer->msgs == msgs );
 
-    tr_publisherPublish( msgs->publisher, msgs->peer, e );
+    tr_publisherPublish( &msgs->publisher, msgs->peer, e );
 }
 
@@ -2210 +2210 @@
 
     m = tr_new0( tr_peermsgs, 1 );
-    m->publisher = tr_publisherNew( );
+    m->publisher = TR_PUBLISHER_INIT;
     m->peer = peer;
     m->session = torrent->session;
@@ -2230 +2230 @@
     peer->msgs = m;
 
-    *setme = tr_publisherSubscribe( m->publisher, func, userData );
+    *setme = tr_publisherSubscribe( &m->publisher, func, userData );
 
     if( tr_peerIoSupportsLTEP( peer->io ) )
@@ -2249 +2249 @@
     {
         tr_timerFree( &msgs->pexTimer );
-        tr_publisherFree( &msgs->publisher );
+        tr_publisherDestruct( &msgs->publisher );
         reqListClear( &msgs->clientWillAskFor );
         reqListClear( &msgs->clientAskedFor );
@@ -2268 +2268 @@
                         tr_publisher_tag tag )
 {
-    tr_publisherUnsubscribe( peer->publisher, tag );
-}
-
+    tr_publisherUnsubscribe( &peer->publisher, tag );
+}
+