cs-education / sysbuild

A system programming learning environment in the browser
https://cs-education.github.io/sys/
Other
36 stars 35 forks source link

Filesystem-int delete non-existing file fix #151

Closed coltonmercurio closed 8 years ago

coltonmercurio commented 8 years ago

Fix odd case where the VM would quickly create a file then delete it. Race condition where the browser filesystem does not have enough time to create a local file before the VM notifies the browser file system to delete it. Fixed by null checking and re-ordering logic.

neelabhg commented 8 years ago

Changes look good to me, although I'm not familiar with the filesystem implementation very much (in case you were waiting on my review)