AugustoRuiz / WYZTracker

A .NET Tracker to create music for WYZPlayer.
MIT License
45 stars 8 forks source link

Optimizaciones y sintaxis tniASM #3

Closed theNestruo closed 7 years ago

theNestruo commented 7 years ago

Incluido Prueba_MSX Corrección de versión en el comentario Optimizaciones menores de código y de usabilidad (e.g.: búferes inicializados en el código del replayer) Versión con sintaxis tniASM

AugustoRuiz commented 7 years ago

Hi!

Thank you very much for the changes! We should make the changes in all player versions (that is, MSX - RAM/ROM, ZX anc CPC). Can you do that?

theNestruo commented 7 years ago

Hi!

I can try doing the changes in all the player versions. Unfortunately I don't have any toolchain for ZX or CPC; meaning that I won't be able to test them. If you can test them, I can upload them to my branch.

AugustoRuiz commented 7 years ago

That would be great! Thank you!

On Mon, Jun 19, 2017 at 10:08 AM, Néstor Sancho notifications@github.com wrote:

Hi!

I can try doing the changes in all the player versions. Unfortunately I don't have any toolchain for ZX or CPC; meaning that I won't be able to test them. If you can test them, I can upload them to my branch.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/AugustoRuiz/WYZTracker/pull/3#issuecomment-309368782, or mute the thread https://github.com/notifications/unsubscribe-auth/ANE5qBnYhw6DN8vO5Px0IcjcU05BOHJQks5sFiyBgaJpZM4NhjuM .

theNestruo commented 7 years ago

Hi! I've propagated the changes to CPC and ZX players. I'm quite confident about the ZX version, but not about the CPC version (it is too different to the other ones). Please test the CPC and ZX players to ensure the changes are correct. Thank you!

Also please note that is no longer required to maintain two versions (ROM/RAM) for MSX; the same source is valid for both scenarios. The *_RAM file simply contains the variables and should be assembled in RAM, but the replayer routines can be assembled in ROM or in RAM.

AugustoRuiz commented 7 years ago

Thank you very much!