Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unload when a new post processor is added. #104

Merged
merged 4 commits into from
Nov 19, 2013

Conversation

xr1337
Copy link
Contributor

@xr1337 xr1337 commented Nov 18, 2013

When TyphoonFactory has had its first component extracted, any future patcher will not be able to influence the get component.

Enforced by unloading when a new post processor is added.

…d all its components.

- though, not a really need implementation as all the components will have to be unloaded.
jasperblues added a commit that referenced this pull request Nov 19, 2013
Unload when a new post processor is added.
@jasperblues jasperblues merged commit 56fc367 into appsquickly:master Nov 19, 2013
@jasperblues
Copy link
Member

@xr1337 Thanks Sufiyan! . . perhaps we should issue a warning to the console here too?

@xr1337
Copy link
Contributor Author

xr1337 commented Nov 19, 2013

Oh that sounds great. 

Will add it in. 


Sent from Mailbox for iPhone

On Tue, Nov 19, 2013 at 1:55 AM, Jasper Blues [email protected]
wrote:

@xr1337 Thanks Sufiyan! . . perhaps we should issue a warning to the console here too?

Reply to this email directly or view it on GitHub:
#104 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants