Changeset 7525 for trunk/libtransmission/utils.c

Timestamp:

Dec 29, 2008, 9:51:54 AM (13 years ago)

Author:

charles

Message:

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

File:

• trunk/libtransmission/utils.c (modified) (2 diffs)

trunk/libtransmission/utils.c

@@ -747 +747 @@
 
 tr_bitfield*
+tr_bitfieldConstruct( tr_bitfield * b, size_t bitCount )
+{
+    b->bitCount = bitCount;
+    b->byteCount = ( bitCount + 7u ) / 8u;
+    b->bits = tr_new0( uint8_t, b->byteCount );
+    return b;
+}
+
+void
+tr_bitfieldDestruct( tr_bitfield * b )
+{
+    tr_free( b->bits );
+}
+
+tr_bitfield*
 tr_bitfieldNew( size_t bitCount )
 {
-    tr_bitfield * ret = tr_new0( tr_bitfield, 1 );
-
-    ret->bitCount = bitCount;
-    ret->byteCount = ( bitCount + 7u ) / 8u;
-    ret->bits = tr_new0( uint8_t, ret->byteCount );
-    return ret;
+    return tr_bitfieldConstruct( tr_new0( tr_bitfield, 1 ), bitCount );
 }
 
@@ -773 +783 @@
     if( bitfield )
     {
-        tr_free( bitfield->bits );
+        tr_bitfieldDestruct( bitfield );
         tr_free( bitfield );
     }