wulfcode icon indicating copy to clipboard operation
wulfcode copied to clipboard
verified publisher icon echolevel

Refactor

Open mxdxgx opened this issue 9 years ago • 4 comments

Extract inner classes from wulfcode.java and encapsulate data. @echolevel that cool with you ?

mxdxgx avatar May 08 '16 17:05 mxdxgx

I'm trying it out on linux, btw. ;)

mxdxgx avatar May 08 '16 17:05 mxdxgx

Great, keen to hear if it works! Sure, any improvements are welcome. I'm experimenting with a Node/Web Audio/Web MIDI version (documenting my progress here), but I aim to keep improving the Java version too.

echolevel avatar May 08 '16 18:05 echolevel

Would be great to get some unit testing in with the refactoring effort. I've not properly analysed the codebase yet but a few tests wouldn't do any harm.

2bard avatar May 09 '16 19:05 2bard

Agreed,I'll add this as a subtask or something (I'm used to jira...lol) :P

mxdxgx avatar May 09 '16 21:05 mxdxgx