Changeset 243 for vtcross/branches/fixingCBR/src
- Timestamp:
- 04/08/09 18:09:40 (15 years ago)
- Files:
-
- 1 modified
Legend:
- Unmodified
- Added
- Removed
-
vtcross/branches/fixingCBR/src/include/vtcross/cbr.h
r241 r243 41 41 42 42 /* TODO This is a terribly hackish way to go about this. Why not just use actual 43 * pointers? Typedefing the pointer out makes the code misleading and hard to44 * understand.43 * pointers? Typedefing the pointer out makes the code misleading and extremely 44 * difficult to understand. 45 45 */ 46 46 typedef cbr_s * cbr; 47 47 48 48 49 /* TODO I don't think these functions do what you want them to do. The 50 * cbr_create is returning an object by value, which means the object that is 51 * being malloc'd in the cbr_create function is effectively a memory leak, since 52 * you lose the reference to that memory, and instead return a copy of the data 53 * in that memory space. 49 /* TODO 54 50 */ 55 51 cbr cbr_create(char * _filename, char * _tablename, char * _cols[], \