diff options
| author | Mike Buland <eichlan@xagasoft.com> | 2009-08-22 04:00:54 +0000 |
|---|---|---|
| committer | Mike Buland <eichlan@xagasoft.com> | 2009-08-22 04:00:54 +0000 |
| commit | 6d4a2e8f5153eefcada061d346aa5d852f2500ee (patch) | |
| tree | a65dbe1ef11d3fae1b3d049c4df3d2dcec4face2 | |
| parent | 78499c848a9c2bafe1db1ec7ceaf8556e2d7c7cc (diff) | |
| download | libbu++-6d4a2e8f5153eefcada061d346aa5d852f2500ee.tar.gz libbu++-6d4a2e8f5153eefcada061d346aa5d852f2500ee.tar.bz2 libbu++-6d4a2e8f5153eefcada061d346aa5d852f2500ee.tar.xz libbu++-6d4a2e8f5153eefcada061d346aa5d852f2500ee.zip | |
Pretty much just removed the debugging from SharedCore, it wasn't at fault, and
it passed many more valgrind tests.
Diffstat (limited to '')
| -rw-r--r-- | src/sharedcore.cpp | 67 | ||||
| -rw-r--r-- | src/sharedcore.h | 37 |
2 files changed, 2 insertions, 102 deletions
diff --git a/src/sharedcore.cpp b/src/sharedcore.cpp index 24d8972..6333335 100644 --- a/src/sharedcore.cpp +++ b/src/sharedcore.cpp | |||
| @@ -7,70 +7,3 @@ | |||
| 7 | 7 | ||
| 8 | #include "bu/sharedcore.h" | 8 | #include "bu/sharedcore.h" |
| 9 | 9 | ||
| 10 | void hardlog(const char *fmt, ...) | ||
| 11 | { | ||
| 12 | va_list ap; | ||
| 13 | va_start( ap, fmt ); | ||
| 14 | |||
| 15 | FILE *fh = fopen("sharedcore.log", "ab"); | ||
| 16 | vfprintf( fh, fmt, ap ); | ||
| 17 | va_end( ap ); | ||
| 18 | fclose( fh ); | ||
| 19 | } | ||
| 20 | |||
| 21 | int iDepth = 0; | ||
| 22 | |||
| 23 | void fncin( void *base, const char *fn, void *core, int *iRefCount, int params, ... ) | ||
| 24 | { | ||
| 25 | va_list ap; | ||
| 26 | va_start( ap, params ); | ||
| 27 | |||
| 28 | FILE *fh = fopen("sharedcore.log", "ab"); | ||
| 29 | fprintf( fh, "%08X: ", base ); | ||
| 30 | for( int j = 0; j < iDepth; j++ ) | ||
| 31 | fprintf( fh, " " ); | ||
| 32 | fprintf( fh, "%s(", fn ); | ||
| 33 | for( int j = 0; j < params; j++ ) | ||
| 34 | if( j == 0 ) | ||
| 35 | fprintf( fh, " %08X", va_arg( ap, void * ) ); | ||
| 36 | else | ||
| 37 | fprintf( fh, ", %08X", va_arg( ap, void * ) ); | ||
| 38 | if( params > 0 ) | ||
| 39 | fprintf( fh, " )"); | ||
| 40 | else | ||
| 41 | fprintf( fh, ")"); | ||
| 42 | |||
| 43 | fprintf( fh, " [%08X / %08X] (%d)\n", core, iRefCount, rc ); | ||
| 44 | va_end( ap ); | ||
| 45 | fclose( fh ); | ||
| 46 | |||
| 47 | iDepth++; | ||
| 48 | } | ||
| 49 | |||
| 50 | void fncout( void *base, const char *fn, void *core, int *iRefCount, int params, ... ) | ||
| 51 | { | ||
| 52 | va_list ap; | ||
| 53 | va_start( ap, params ); | ||
| 54 | |||
| 55 | iDepth--; | ||
| 56 | |||
| 57 | FILE *fh = fopen("sharedcore.log", "ab"); | ||
| 58 | fprintf( fh, "%08X: ", base ); | ||
| 59 | for( int j = 0; j < iDepth; j++ ) | ||
| 60 | fprintf( fh, " " ); | ||
| 61 | fprintf( fh, "%s(", fn ); | ||
| 62 | for( int j = 0; j < params; j++ ) | ||
| 63 | if( j == 0 ) | ||
| 64 | fprintf( fh, " %08X", va_arg( ap, void * ) ); | ||
| 65 | else | ||
| 66 | fprintf( fh, ", %08X", va_arg( ap, void * ) ); | ||
| 67 | if( params > 0 ) | ||
| 68 | fprintf( fh, " )"); | ||
| 69 | else | ||
| 70 | fprintf( fh, ")"); | ||
| 71 | |||
| 72 | fprintf( fh, " [%08X / %08X] (%d)\n", core, iRefCount, rc ); | ||
| 73 | va_end( ap ); | ||
| 74 | fclose( fh ); | ||
| 75 | } | ||
| 76 | |||
diff --git a/src/sharedcore.h b/src/sharedcore.h index 7960fa2..046e973 100644 --- a/src/sharedcore.h +++ b/src/sharedcore.h | |||
| @@ -13,17 +13,6 @@ | |||
| 13 | #include <stdio.h> | 13 | #include <stdio.h> |
| 14 | #include <stdarg.h> | 14 | #include <stdarg.h> |
| 15 | 15 | ||
| 16 | void hardlog(const char *fmt, ...); | ||
| 17 | |||
| 18 | void fncin( void *base, const char *fn, void *core, int *refcnt, int params, ... ); | ||
| 19 | void fncout( void *base, const char *fn, void *core, int *refcnt, int params, ... ); | ||
| 20 | |||
| 21 | |||
| 22 | #define fin( count, ... ) fncin( (void *)this, __FUNCTION__, core, iRefCount, count, ##__VA_ARGS__ ) | ||
| 23 | #define fout( count, ... ) fncout( (void *)this, __FUNCTION__, core, iRefCount, count, ##__VA_ARGS__ ) | ||
| 24 | |||
| 25 | #define rc ((iRefCount==NULL)?(-1):(*iRefCount)) | ||
| 26 | |||
| 27 | namespace Bu | 16 | namespace Bu |
| 28 | { | 17 | { |
| 29 | template<typename Core> | 18 | template<typename Core> |
| @@ -35,33 +24,25 @@ namespace Bu | |||
| 35 | core( NULL ), | 24 | core( NULL ), |
| 36 | iRefCount( NULL ) | 25 | iRefCount( NULL ) |
| 37 | { | 26 | { |
| 38 | fin( 0 ); | ||
| 39 | core = _allocateCore(); | 27 | core = _allocateCore(); |
| 40 | iRefCount = new int(1); | 28 | iRefCount = new int(1); |
| 41 | fout( 0 ); | ||
| 42 | } | 29 | } |
| 43 | 30 | ||
| 44 | SharedCore( const _SharedType &rSrc ) : | 31 | SharedCore( const _SharedType &rSrc ) : |
| 45 | core( NULL ), | 32 | core( NULL ), |
| 46 | iRefCount( NULL ) | 33 | iRefCount( NULL ) |
| 47 | { | 34 | { |
| 48 | fin( 1, &rSrc ); | ||
| 49 | _softCopy( rSrc ); | 35 | _softCopy( rSrc ); |
| 50 | fout( 1, &rSrc ); | ||
| 51 | } | 36 | } |
| 52 | 37 | ||
| 53 | virtual ~SharedCore() | 38 | virtual ~SharedCore() |
| 54 | { | 39 | { |
| 55 | fin( 0 ); | ||
| 56 | _deref(); | 40 | _deref(); |
| 57 | fout( 0 ); | ||
| 58 | } | 41 | } |
| 59 | 42 | ||
| 60 | SharedCore &operator=( const SharedCore &rhs ) | 43 | SharedCore &operator=( const SharedCore &rhs ) |
| 61 | { | 44 | { |
| 62 | fin( 1, &rhs ); | ||
| 63 | _softCopy( rhs ); | 45 | _softCopy( rhs ); |
| 64 | fout( 1, &rhs ); | ||
| 65 | return *this; | 46 | return *this; |
| 66 | } | 47 | } |
| 67 | 48 | ||
| @@ -74,43 +55,34 @@ namespace Bu | |||
| 74 | Core *core; | 55 | Core *core; |
| 75 | void _hardCopy() | 56 | void _hardCopy() |
| 76 | { | 57 | { |
| 77 | fin( 0 ); | ||
| 78 | if( !core || !iRefCount ) | 58 | if( !core || !iRefCount ) |
| 79 | { fout( 0 ); return; } | 59 | return; |
| 80 | if( (*iRefCount) == 1 ) | 60 | if( (*iRefCount) == 1 ) |
| 81 | { fout( 0 ); return; } | 61 | return; |
| 82 | Core *copy = _copyCore( core ); | 62 | Core *copy = _copyCore( core ); |
| 83 | _deref(); | 63 | _deref(); |
| 84 | core = copy; | 64 | core = copy; |
| 85 | iRefCount = new int( 1 ); | 65 | iRefCount = new int( 1 ); |
| 86 | fout( 0 ); | ||
| 87 | } | 66 | } |
| 88 | 67 | ||
| 89 | virtual Core *_allocateCore() | 68 | virtual Core *_allocateCore() |
| 90 | { | 69 | { |
| 91 | fin( 0 ); | ||
| 92 | fout( 0 ); | ||
| 93 | return new Core(); | 70 | return new Core(); |
| 94 | } | 71 | } |
| 95 | 72 | ||
| 96 | virtual Core *_copyCore( Core *pSrc ) | 73 | virtual Core *_copyCore( Core *pSrc ) |
| 97 | { | 74 | { |
| 98 | fin( 0 ); | ||
| 99 | fout( 0 ); | ||
| 100 | return new Core( *pSrc ); | 75 | return new Core( *pSrc ); |
| 101 | } | 76 | } |
| 102 | 77 | ||
| 103 | virtual void _deallocateCore( Core *pSrc ) | 78 | virtual void _deallocateCore( Core *pSrc ) |
| 104 | { | 79 | { |
| 105 | fin( 0 ); | ||
| 106 | fout( 0 ); | ||
| 107 | delete pSrc; | 80 | delete pSrc; |
| 108 | } | 81 | } |
| 109 | 82 | ||
| 110 | private: | 83 | private: |
| 111 | void _deref() | 84 | void _deref() |
| 112 | { | 85 | { |
| 113 | fin( 0 ); | ||
| 114 | if( (--(*iRefCount)) == 0 ) | 86 | if( (--(*iRefCount)) == 0 ) |
| 115 | { | 87 | { |
| 116 | _deallocateCore( core ); | 88 | _deallocateCore( core ); |
| @@ -118,26 +90,21 @@ namespace Bu | |||
| 118 | } | 90 | } |
| 119 | core = NULL; | 91 | core = NULL; |
| 120 | iRefCount = NULL; | 92 | iRefCount = NULL; |
| 121 | fout( 0 ); | ||
| 122 | } | 93 | } |
| 123 | 94 | ||
| 124 | void _incRefCount() | 95 | void _incRefCount() |
| 125 | { | 96 | { |
| 126 | fin( 0 ); | ||
| 127 | if( iRefCount && core ) | 97 | if( iRefCount && core ) |
| 128 | ++(*iRefCount); | 98 | ++(*iRefCount); |
| 129 | fout( 0 ); | ||
| 130 | } | 99 | } |
| 131 | 100 | ||
| 132 | void _softCopy( const _SharedType &rSrc ) | 101 | void _softCopy( const _SharedType &rSrc ) |
| 133 | { | 102 | { |
| 134 | fin( 1, &rSrc ); | ||
| 135 | if( core ) | 103 | if( core ) |
| 136 | _deref(); | 104 | _deref(); |
| 137 | core = rSrc.core; | 105 | core = rSrc.core; |
| 138 | iRefCount = rSrc.iRefCount; | 106 | iRefCount = rSrc.iRefCount; |
| 139 | _incRefCount(); | 107 | _incRefCount(); |
| 140 | fout( 1, &rSrc ); | ||
| 141 | } | 108 | } |
| 142 | 109 | ||
| 143 | int *iRefCount; | 110 | int *iRefCount; |
