search

reactphp / filesystem

Evented filesystem access.
MIT License
135 stars 38 forks source link

Remove call invokers #61

Open WyriHaximus opened 5 years ago

WyriHaximus commented 5 years ago

Initially these call invokers where added to keep ext-eio under control. And they made sense in the design for that adapter. But with uv adapter coming up this feels less and less like it makes sense due to the increased overhead caused by it.

WyriHaximus commented 5 years ago

Ping @clue @CharlotteDunois what do you think?

clue commented 5 years ago

Makes perfect sense! I understand where they're coming from, but I agree that they feel more like a burden than actually helping with implementing new features in the current state :+1:

WyriHaximus commented 5 years ago

I'll get onto it :+1: