]> git.sesse.net Git - vlc/commitdiff
Revert "Remove block_Realloc() shrinking behavior"
authorRémi Denis-Courmont <remi@remlab.net>
Tue, 29 Sep 2009 19:36:42 +0000 (22:36 +0300)
committerRémi Denis-Courmont <remi@remlab.net>
Tue, 29 Sep 2009 19:36:42 +0000 (22:36 +0300)
This reverts commit fd4cb4889d55be7d30e3d85cfb406b2814c1655e.

We don't need this commit if we don't use posix_memalign().

src/misc/block.c

index ecc0c580bc503183b9076c93380d8fd87ddaf219..77800d6a2f7f6bdae7ad14a00bbb7b93e4d1d1cc 100644 (file)
@@ -93,6 +93,8 @@ static void BlockMetaCopy( block_t *restrict out, const block_t *in )
 #define BLOCK_ALIGN        16
 /* Initial reserved header and footer size (must be multiple of alignment) */
 #define BLOCK_PADDING      32
+/* Maximum size of reserved footer before we release with realloc() */
+#define BLOCK_WASTE_SIZE   2048
 
 block_t *block_Alloc( size_t i_size )
 {
@@ -225,6 +227,23 @@ block_t *block_Realloc( block_t *p_block, ssize_t i_prebody, size_t i_body )
         block_Release( p_block );
         p_block = p_rea;
     }
+    else
+    /* We have a very large reserved footer now? Release some of it.
+     * XXX it might not preserve the alignment of p_buffer */
+    if( p_end - (p_block->p_buffer + i_body) > BLOCK_WASTE_SIZE )
+    {
+        const ptrdiff_t i_prebody = p_block->p_buffer - p_start;
+        const size_t i_new = requested + 1 * BLOCK_PADDING;
+        block_sys_t *p_new = realloc( p_sys, sizeof (*p_sys) + i_new );
+
+        if( p_new != NULL )
+        {
+            p_sys = p_new;
+            p_sys->i_allocated_buffer = i_new;
+            p_block = &p_sys->self;
+            p_block->p_buffer = &p_sys->p_allocated_buffer[i_prebody];
+        }
+    }
 
     /* NOTE: p_start and p_end are corrupted from this point */