Todo List

Member AVIFile::AVIFile ()

mainHdr not initialized

add checking for NULL pointers

Member AVIFile::AVIFile (const AVIFile &)

add checking for NULL pointers

Member AVIFile::GetDVFrame (Frame &frame, int frameNum)

we actually don't need the frame here, we could use just a void pointer

Parameters:

	frame	a reference to the frame object that will receive the frame data
	frameNum	the frame number to read

Returns:

0 if the frame could be read, -1 otherwise

Member AVIFile::GetDVFrameInfo (off_t &offset, int &size, int frameNum)

the size parameter is redundant.

all index related operations should be isolated

Parameters:

	offset	the file offset to the start of the frame
	size	the size of the frame
	frameNum	the number of the frame we wish to find

Returns:

0 if the frame could be found, -1 otherwise

Member AVIFile::GetFrameInfo (off_t &offset, int &size, int frameNum, FOURCC chunkID)

the size parameter is redundant.

all index related operations should be isolated

Parameters:

	offset	the file offset to the start of the frame
	size	the size of the frame
	frameNum	the number of the frame we wish to find
	chunkID	the ID of the type of chunk we want

Returns:

0 if the frame could be found, -1 otherwise

Member AVIFile::Init (int format, int sampleFrequency, int indexType)

consolidate AVIFile::Init, AVI1File::Init, AVI2File::Init.

Member AVIFile::PrintDirectoryEntryData (const RIFFDirEntry &entry) const

use 64 bit routines

Parameters:

entry

the entry to print

Member Frame::IsPAL (void) const

function can't handle "empty" frame

Returns:

true for PAL frame, false for a NTSC frame

Member RIFFFile::FindDirectoryEntry (FOURCC type, int n=0) const

inefficient if the directory has lots of items

Parameters:

	type	the type of the entry to find
	n	the zero-based instance of type to locate

Returns:

the index of the found object in the directory, or -1 if not found

---