aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDiego Biurrun <diego@biurrun.de>2012-10-18 19:16:37 +0200
committerDiego Biurrun <diego@biurrun.de>2012-10-20 02:34:29 +0200
commit5532cf3178385dc7d5414b65e9f01966125d8a9b (patch)
tree195fc33e4054e617808cebc97fac4872b0ceb76f
parentc08536979be1a4e9a29e3ee7a6b79f6d73bb12d2 (diff)
downloadffmpeg-5532cf3178385dc7d5414b65e9f01966125d8a9b.tar.gz
avutil/mem: K&R formatting cosmetics
-rw-r--r--libavutil/mem.c77
-rw-r--r--libavutil/mem.h4
2 files changed, 41 insertions, 40 deletions
diff --git a/libavutil/mem.c b/libavutil/mem.c
index 3769da7d21..16c1adb014 100644
--- a/libavutil/mem.c
+++ b/libavutil/mem.c
@@ -58,8 +58,8 @@ void free(void *ptr);
#endif /* MALLOC_PREFIX */
/* You can redefine av_malloc and av_free in your project to use your
- memory allocator. You do not need to suppress this file because the
- linker will do it automatically. */
+ * memory allocator. You do not need to suppress this file because the
+ * linker will do it automatically. */
void *av_malloc(size_t size)
{
@@ -69,46 +69,46 @@ void *av_malloc(size_t size)
#endif
/* let's disallow possible ambiguous cases */
- if (size > (INT_MAX-32) || !size)
+ if (size > (INT_MAX - 32) || !size)
return NULL;
#if CONFIG_MEMALIGN_HACK
- ptr = malloc(size+32);
- if(!ptr)
+ ptr = malloc(size + 32);
+ if (!ptr)
return ptr;
- diff= ((-(long)ptr - 1)&31) + 1;
- ptr = (char*)ptr + diff;
- ((char*)ptr)[-1]= diff;
+ diff = ((-(long)ptr - 1) & 31) + 1;
+ ptr = (char *)ptr + diff;
+ ((char *)ptr)[-1] = diff;
#elif HAVE_POSIX_MEMALIGN
- if (posix_memalign(&ptr,32,size))
+ if (posix_memalign(&ptr, 32, size))
ptr = NULL;
#elif HAVE_ALIGNED_MALLOC
ptr = _aligned_malloc(size, 32);
#elif HAVE_MEMALIGN
- ptr = memalign(32,size);
+ ptr = memalign(32, size);
/* Why 64?
- Indeed, we should align it:
- on 4 for 386
- on 16 for 486
- on 32 for 586, PPro - K6-III
- on 64 for K7 (maybe for P3 too).
- Because L1 and L2 caches are aligned on those values.
- But I don't want to code such logic here!
+ * Indeed, we should align it:
+ * on 4 for 386
+ * on 16 for 486
+ * on 32 for 586, PPro - K6-III
+ * on 64 for K7 (maybe for P3 too).
+ * Because L1 and L2 caches are aligned on those values.
+ * But I don't want to code such logic here!
*/
- /* Why 32?
- For AVX ASM. SSE / NEON needs only 16.
- Why not larger? Because I did not see a difference in benchmarks ...
+ /* Why 32?
+ * For AVX ASM. SSE / NEON needs only 16.
+ * Why not larger? Because I did not see a difference in benchmarks ...
*/
- /* benchmarks with P3
- memalign(64)+1 3071,3051,3032
- memalign(64)+2 3051,3032,3041
- memalign(64)+4 2911,2896,2915
- memalign(64)+8 2545,2554,2550
- memalign(64)+16 2543,2572,2563
- memalign(64)+32 2546,2545,2571
- memalign(64)+64 2570,2533,2558
-
- BTW, malloc seems to do 8-byte alignment by default here.
+ /* benchmarks with P3
+ * memalign(64) + 1 3071, 3051, 3032
+ * memalign(64) + 2 3051, 3032, 3041
+ * memalign(64) + 4 2911, 2896, 2915
+ * memalign(64) + 8 2545, 2554, 2550
+ * memalign(64) + 16 2543, 2572, 2563
+ * memalign(64) + 32 2546, 2545, 2571
+ * memalign(64) + 64 2570, 2533, 2558
+ *
+ * BTW, malloc seems to do 8-byte alignment by default here.
*/
#else
ptr = malloc(size);
@@ -123,14 +123,15 @@ void *av_realloc(void *ptr, size_t size)
#endif
/* let's disallow possible ambiguous cases */
- if(size > (INT_MAX-16) )
+ if (size > (INT_MAX - 16))
return NULL;
#if CONFIG_MEMALIGN_HACK
//FIXME this isn't aligned correctly, though it probably isn't needed
- if(!ptr) return av_malloc(size);
- diff= ((char*)ptr)[-1];
- return (char*)realloc((char*)ptr - diff, size + diff) + diff;
+ if (!ptr)
+ return av_malloc(size);
+ diff = ((char *)ptr)[-1];
+ return (char *)realloc((char *)ptr - diff, size + diff) + diff;
#elif HAVE_ALIGNED_MALLOC
return _aligned_realloc(ptr, size, 32);
#else
@@ -142,7 +143,7 @@ void av_free(void *ptr)
{
#if CONFIG_MEMALIGN_HACK
if (ptr)
- free((char*)ptr - ((char*)ptr)[-1]);
+ free((char *)ptr - ((char *)ptr)[-1]);
#elif HAVE_ALIGNED_MALLOC
_aligned_free(ptr);
#else
@@ -152,7 +153,7 @@ void av_free(void *ptr)
void av_freep(void *arg)
{
- void **ptr= (void**)arg;
+ void **ptr = (void **)arg;
av_free(*ptr);
*ptr = NULL;
}
@@ -167,8 +168,8 @@ void *av_mallocz(size_t size)
char *av_strdup(const char *s)
{
- char *ptr= NULL;
- if(s){
+ char *ptr = NULL;
+ if (s) {
int len = strlen(s) + 1;
ptr = av_malloc(len);
if (ptr)
diff --git a/libavutil/mem.h b/libavutil/mem.h
index fb222061f7..4f14f27a91 100644
--- a/libavutil/mem.h
+++ b/libavutil/mem.h
@@ -89,7 +89,7 @@ void *av_malloc(size_t size) av_malloc_attrib av_alloc_size(1);
* be allocated.
* @see av_malloc()
*/
-av_alloc_size(1,2) static inline void *av_malloc_array(size_t nmemb, size_t size)
+av_alloc_size(1, 2) static inline void *av_malloc_array(size_t nmemb, size_t size)
{
if (size <= 0 || nmemb >= INT_MAX / size)
return NULL;
@@ -140,7 +140,7 @@ void *av_mallocz(size_t size) av_malloc_attrib av_alloc_size(1);
* @see av_mallocz()
* @see av_malloc_array()
*/
-av_alloc_size(1,2) static inline void *av_mallocz_array(size_t nmemb, size_t size)
+av_alloc_size(1, 2) static inline void *av_mallocz_array(size_t nmemb, size_t size)
{
if (size <= 0 || nmemb >= INT_MAX / size)
return NULL;