Help save net neutrality! Learn more.
Close

#74 ImplAAFRandomRawStorage::SetExtent(aafUint64 extent)

closed
nobody
None
5
2004-03-03
2004-01-15
Jim Trainor
No

ImplAAFRandomRawStorage::SetExtent(aafUint64
extent) includes the following test:

if (extent > ((unsigned long) ~0))
return AAFRESULT_DATA_SIZE;

i.e. refuses to set the file extent (size) greater than 32
bits.

This is unnecessary in the brave new (well... not so
new) world of files greater than 4 GB in size.

I removed this in the schemasoft branch. The bug
should be closes when that branch is merged into the
main branch.

(Note, I stumbled over this only after implementing
OMStream::size() using _fstati64() rather of than the
non-64-bit-capable Windows fseek call, and writing test
files > 4GB in size. That change is uncommited since it
may not be an adequate solution to the OMStream::size
() problem.)

Discussion

  • Jim Trainor

    Jim Trainor - 2004-03-03
    • status: open --> closed
     
  • Jim Trainor

    Jim Trainor - 2004-03-03

    Logged In: YES
    user_id=239292

    The offending fragment of code was removed in the main
    branch and subsequently merged into the main branch.