I think it ends up being a lot more convenient and fast to store actual Inodes as the children in InodeFolder rather than Inode id's, so we end up needing to look up the inode associated with the id's from the input stream here, which is why we need the map. I guess we could store a reference to mInodes in InodeFolder, but that would be kind of messy.