search

santiontanon / mdlz80optimizer

MDL (a Z80 assembler optimizer)
Apache License 2.0
81 stars 6 forks source link

VSCode integration (+logging, +new pattern...) #3

Closed theNestruo closed 4 years ago

theNestruo commented 4 years ago

This code should fix the "include path" problem. New pattern ("Replace call nn; ret with jp nn"). Proper logging library (Logback) for performance and flexibility. VSCode integration (see picture!). More "human readable" pattern names (mainly for VSCode integration).

VSCode integration

santiontanon commented 4 years ago

Thanks Nestor! Woooww! That VSCode integration looks really cool!!!! I will install VSCode tonight to try it out! Thanks a lot for all the renaming btw, that must have taken a while!!

Just resolved all the conflicts, and merging right now! I hope you don't mind if I revert that Singleton pattern though. Singletons is something that have been the source of many a problem in the past in my experience, and I have really gone out of the way to avoid using it (notice how I pass MDLConfig around everywhere, rather than using a Singleton).