-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Great job! #1
Comments
Hey @dbousamra. Thanks for the message! Indeed it's been really an adventure writing this. The funny thing is I considered doing this in Haskell! I did the advent of code challenge in Haskell last year and had a lot of fun. In the end I wans't as brave as you, though. Really nice work there too! |
I agree: Great job, @rbaron. Just because of my OSD, I must comment: Constant names follow the same naming rule of a normal variable or type in Go: |
Amazing. Thanks for the tip @jairodasilva. I'll take a closer look at gometalinter! About the code, when I began writing this, I didin't have a very good mental model of all the moving parts of the NES. What helped me a lot was reading the source code for several other open source emulators out there. In particular, I messed around a lot with AndreaOrru/LaiNES and fogleman/nes. The first one is amazing in that it is super concise and the info on nesdev wiki is often straightforwardly distilled in the code. The second one is an extremely well written and polished NES emulator in Go. I highly recommend taking a look at both! |
Hi
I'm not a Lobster's member :( Github needs to allow comments. I've written a NES emulator (http://github.com/dbousamra/hnes), so I know how tricky the NES is. I found your code really clean, and easy to understand. Nice work!
The text was updated successfully, but these errors were encountered: