author | zecke <zecke> | 2004-09-18 16:43:47 (UTC) |
---|---|---|
committer | zecke <zecke> | 2004-09-18 16:43:47 (UTC) |
commit | c81e90d0f6c0feec1443d711833ea382a1c10ec6 (patch) (side-by-side diff) | |
tree | 7390346761cfbceee0abccb51a80e30898f54f3b | |
parent | fa5c8471c3053dfc8d9742426faa569b3aa03d71 (diff) | |
download | opie-c81e90d0f6c0feec1443d711833ea382a1c10ec6.zip opie-c81e90d0f6c0feec1443d711833ea382a1c10ec6.tar.gz opie-c81e90d0f6c0feec1443d711833ea382a1c10ec6.tar.bz2 |
A proposal for making the memory copy functions build with g++3.4.2.
Please review the patch, specially the casts it makes, and also
the boundaries and sizes.
-Shouldn't we use unsigned char* instead of char*. For arm it is already unsigned?
-Check again the casts
-rw-r--r-- | noncore/multimedia/opieplayer2/xinevideowidget.cpp | 66 |
1 files changed, 47 insertions, 19 deletions
diff --git a/noncore/multimedia/opieplayer2/xinevideowidget.cpp b/noncore/multimedia/opieplayer2/xinevideowidget.cpp index b55750a..15c611f 100644 --- a/noncore/multimedia/opieplayer2/xinevideowidget.cpp +++ b/noncore/multimedia/opieplayer2/xinevideowidget.cpp @@ -48,10 +48,28 @@ -static inline void memcpy_rev ( void *dst, void *src, size_t len ) +/* + * This code relies the len be a multiply of 16bit + */ +static inline void memcpy_rev ( void *_dst, void *_src, size_t len ) { - len >>= 1; - - ((char *) src ) += ( len << 1 ); + /* + * move the source to the end + */ + char *src_c = static_cast<char*>(_src) + len; + + /* + * as we copy by 16bit and not 8bit + * devide the length by two + */ + len >>= 1; + + short int* dst = static_cast<short int*>( _dst ); + short int* src = reinterpret_cast<short int*>( src_c ); + + /* + * Increment dst after assigning + * Decrement src before assigning becase we move backwards + */ + while ( len-- ) + *dst++ = *--src; - while ( len-- ) - *((short int *) dst )++ = *--((short int *) src ); } @@ -60,9 +78,16 @@ static inline void memcpy_rev ( void *dst, void *src, size_t len ) -static inline void memcpy_step ( void *dst, void *src, size_t len, size_t step ) +static inline void memcpy_step ( void *_dst, void *_src, size_t len, size_t step ) { - len >>= 1; - while ( len-- ) { - *((short int *) dst )++ = *((short int *) src ); - ((char *) src ) += step; - } + short int *dst = static_cast<short int*>( _dst ); + short int *src = static_cast<short int*>( _src ); + + len >>= 1; + + /* + * Copy 16bit from src to dst and move to the next address + */ + while ( len-- ) { + *dst++ = *src; + src = reinterpret_cast<short int*>(reinterpret_cast<char*>(src)+step); + } } @@ -71,12 +96,15 @@ static inline void memcpy_step ( void *dst, void *src, size_t len, size_t step ) -static inline void memcpy_step_rev ( void *dst, void *src, size_t len, size_t step ) +static inline void memcpy_step_rev ( void *_dst, void *_src, size_t len, size_t step ) { - len >>= 1; + len >>= 1; - ((char *) src ) += ( len * step ); + char *src_c = static_cast<char*>( _src ) + (len*step); + short int* dst = static_cast<short int*>( _dst ); + short int* src = reinterpret_cast<short int*>( src_c ); - while ( len-- ) { - ((char *) src ) -= step; - *((short int *) dst )++ = *((short int *) src ); - } + while ( len-- ) { + src_c -= step; + src = reinterpret_cast<short int*>( src_c ); + *dst++ = *src; + } } |