diff options
author | stijn <stinos@zoho.com> | 2014-06-12 17:42:20 +0200 |
---|---|---|
committer | stijn <stinos@zoho.com> | 2014-06-13 20:42:06 +0200 |
commit | f33385f56df29d2c3c65ecb8b68fef6dda5d2b4c (patch) | |
tree | d9c68db52c79ffa2ae4d504a9a6a32bfa6a7b880 | |
parent | 8340c4838948876a4e477556beaed79e59e5e9b4 (diff) | |
download | micropython-f33385f56df29d2c3c65ecb8b68fef6dda5d2b4c.tar.gz micropython-f33385f56df29d2c3c65ecb8b68fef6dda5d2b4c.zip |
gc: Use byte* pointers instead of void* for pointer arithmetic
void* is of unknown size
-rw-r--r-- | py/gc.c | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -120,7 +120,7 @@ void gc_init(void *start, void *end) { // F = A * BLOCKS_PER_ATB / BLOCKS_PER_FTB // P = A * BLOCKS_PER_ATB * BYTES_PER_BLOCK // => T = A * (1 + BLOCKS_PER_ATB / BLOCKS_PER_FTB + BLOCKS_PER_ATB * BYTES_PER_BLOCK) - machine_uint_t total_byte_len = end - start; + machine_uint_t total_byte_len = (byte*)end - (byte*)start; #if MICROPY_ENABLE_FINALISER gc_alloc_table_byte_len = total_byte_len * BITS_PER_BYTE / (BITS_PER_BYTE + BITS_PER_BYTE * BLOCKS_PER_ATB / BLOCKS_PER_FTB + BITS_PER_BYTE * BLOCKS_PER_ATB * BYTES_PER_BLOCK); #else @@ -136,8 +136,8 @@ void gc_init(void *start, void *end) { #endif machine_uint_t gc_pool_block_len = gc_alloc_table_byte_len * BLOCKS_PER_ATB; - gc_pool_start = end - gc_pool_block_len * BYTES_PER_BLOCK; - gc_pool_end = end; + gc_pool_start = (machine_uint_t*)((byte*)end - gc_pool_block_len * BYTES_PER_BLOCK); + gc_pool_end = (machine_uint_t*)end; // clear ATBs memset(gc_alloc_table_start, 0, gc_alloc_table_byte_len); @@ -400,7 +400,7 @@ found: } // get pointer to first block - void *ret_ptr = (void*)(gc_pool_start + start_block * WORDS_PER_BLOCK); + byte *ret_ptr = (byte*)(gc_pool_start + start_block * WORDS_PER_BLOCK); // zero out the additional bytes of the newly allocated blocks // This is needed because the blocks may have previously held pointers @@ -571,7 +571,7 @@ void *gc_realloc(void *ptr_in, machine_uint_t n_bytes) { } // zero out the additional bytes of the newly allocated blocks (see comment above in gc_alloc) - memset(ptr_in + n_bytes, 0, new_blocks * BYTES_PER_BLOCK - n_bytes); + memset((byte*)ptr_in + n_bytes, 0, new_blocks * BYTES_PER_BLOCK - n_bytes); return ptr_in; } |