assimp/assimp — pull request #195 by henrikbuchholz (2013-11-25 11:56:25+00:00)
Title: DefaultIOStream::FileSize(): Replaced windows-based version to use mFile directly

DefaultIOStream::FileSize(): Replaced windows-based version to use mFile directly instead of fileStat

Unfortunately, it is not 100% clear anymore what was the main motivation for this change, because it is several years old and I don't have a changelog of the author.

It might be related to the TODO comment line in 113 in the original AssImp code. Maybe it was faster or it avoided issues in case that file loaders used exclusive open.

All I know is that this change is not the accidental result of a merge: According to SVN blame, this code section had never been touched since revision 1, i.e. the code section was not just an accidental revert to some older state of the assImp repository.

Therefore, it seems pretty likely that it fixed some relevant problem in our product - maybe related to loaders with exclusive open. Also, it is used and stable working in our product for years now.