natevw / fatfs

Standalone FAT16/FAT32 filesystem implementation in JavaScript
47 stars 13 forks source link

Don't "optimistically" parse directory entries #9

Closed natevw closed 10 years ago

natevw commented 10 years ago

Right now we extract everything from every directory entry while iterating. This is one of the things killing performance on Tessel, and is pointless work regardless. We do kinda need the whole long name entries for pretty much all the use cases, but it's actually the main entries that take twice as long to parse (more bitwise fields like dates and times?).

natevw commented 10 years ago

This was [mostly] fixed by https://github.com/natevw/fatfs/commit/37e75c6431875276b20d7c1eaf1f64e85dc31895 [see following bugfix patches].