|
|
Did you ever wonder if OpenALHarness belongs in the CroquetHarness class hierarchy? I wanted to test the sound stuff independently of Croquet, so I made a sound player class and I needed a harness to hold the OpenAL so I copied OpenALHarness out and deleted the voice chat code that was referencing the avatar instance variable. It was easy to do, and I had the impression that it was never designed to fit that way to begin with.
It seems the original CroquetHarness is quite nice with lots of helpful comments and it doesn't seem that a deep hierarchy of classes is the right way to organize this code.
Has anyone been looking at refactoring this? Should I give it a go? Or maybe it's a paragon of clarity and I just haven't yet achieved enlightenment...
Steve
|
Locked
