-
Notifications
You must be signed in to change notification settings - Fork 1
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
Attribute NanoboyAdvance #6
Comments
Sorry about that. The credits got lost while I was cleaning up the readme. I didn't want to be disrespectful. The main influences of my emulator have been GBE+ in the very early and NanoboyAdvance for the later stages. I am quite embarrassed by the RTC situation because I wasted most of my weekend on it without properly understanding it. I had a half working version on a local branch but instead of throwing it away and doing it all over I decided to take your code. I wanted to revisit it this week, fully understand it and make my own thing. |
I'm glad this is settled now and that the lack of attribution was merely an error from reworking the README. |
It always seemed to me that eggvance has taken strong inspiration from NanoboyAdvance from the get go.
The latest case, the RTC implementation, is a clear case to me. The code is pretty much the exact same except for some minor naming and style differences and fewer comments. It follows the exact same structure and logic.
NanoboyAdvance:
https://github.com/fleroviux/NanoboyAdvance/blob/master/source/emulator/cartridge/gpio/rtc.cpp
eggvance:
https://github.com/jsmolka/eggvance/blob/d925e80d3cc11ca043fbdc896524fba787a3226b/eggvance/src/gamepak/rtc.cpp
In the past I've looked over this because a) NanoboyAdvance was credited to some extent in the README (while now any credit is gone) and b) it was less clear cut that some code is a direct copy of mine instead of merely being inspired.
Therefore I kindly ask to add proper attribution.
The text was updated successfully, but these errors were encountered: