Closed Dovyski closed 10 years ago
Is this pull request still in progress, or are you still wanting to add any more commits to it before it is merged?
Sorry I haven't had a chance to have a good luck at your code yet or test it out for myself, but since you have tested it in the Sandbox, when you feel it is at a stable state and you are finished with it, you are welcome to merge it.
I've started working on the record functionality, which I was planning to do after the pull request got merged. I think I will just add the comments/docs I left behind and merge it.
The record functionality will be broken, but at least the new plugin architecture will be ready to be used by everyone on dev
.
I'm done adding commits to this :smile:
The new plugin architecture is stable and ready to be used. I've tested it and it's good to go (excluding FlxReplay
, which is broken and will be fixed in another pull request). I will merge this at the end of the day.
This is pull request 1 of 3 regarding the new plugin system. Here's what I've done so far:
FlxSignal
andFlxG.signals
. There are a couple signals already working (e.g.preUpdate
);TimeManager
andDebugPathDisplay
to use the new plugin architecture;FlxPlugin
) to define the structure of any plugin;What is missing (next tasks):
Please give feedback about the code and the architecture. I've tried to mimic HaxeFlixel API/names/packages when possible. After this pull request gets merged, I will start working on the record plugin.