Implemented Ninia Thread

plasma-umass / Ninia

Python interpreter in JavaScript

Other

20 stars 8 forks source link

Closed Mdrbhatti closed 9 years ago

Mdrbhatti commented 9 years ago

Created a Thread object and refactored Ninia to use it.

jvilk commented 9 years ago

I've added comments. Most are nitpicky stylistic comments. Overall, great work!

Ping me when you've addressed the comments, and I'll look this over and merge if all is well.

Mdrbhatti commented 9 years ago

Done!

Mdrbhatti commented 9 years ago

I have changed the run method to be private.

jvilk commented 9 years ago

Alright! The same tests pass, and seems good to me. Merged.