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

Managed & Unmanaged FBSimulator #21

Merged
merged 1 commit into from
Sep 23, 2015
Merged

Conversation

lawrencelomax
Copy link
Contributor

Extracts out FBSimulator simulator behavior to allow all Simulators to be wrapped with the FBSimulator conveniences that are relevant, regardless of whether they are managed or unmanaged.

A Concreted subclass FBManagedSimulator deals with allocation/freeing of these Simulators. This improves a lot of the inflation of SimDevice to FBSimulator and means that FBSimulatorPool spends a lot less time dealing with heterogeneous objects.

This also means that expensive information about a simulator (for example the construction of processes that have been created through launchd_sim) can be done lazily and will apply to any simulator, not just 'managed' ones.

@mmmulani
Copy link
Contributor

lgtm

lawrencelomax added a commit that referenced this pull request Sep 23, 2015
@lawrencelomax lawrencelomax merged commit deda691 into master Sep 23, 2015
@lawrencelomax lawrencelomax deleted the unmanaged-managed-simulator branch September 23, 2015 11:43
plu added a commit to plu/FBSimulatorControl that referenced this pull request Feb 26, 2017
plu added a commit to plu/FBSimulatorControl that referenced this pull request Feb 26, 2017
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