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

crash when displaying the leaks details #52

Open
menesh2 opened this issue Dec 30, 2018 · 12 comments
Open

crash when displaying the leaks details #52

menesh2 opened this issue Dec 30, 2018 · 12 comments

Comments

@menesh2
Copy link

menesh2 commented Dec 30, 2018

For some reason the app crashes when trying to present the details of memory leaks by pressing the "i" button

screen shot 2018-12-30 at 11 58 58

Why is this happening? am i doing something wrong?

edit:
i changed the version to 1.6 and now it happens when i'm trying to display the leaks

@menesh2 menesh2 changed the title crash when pressing the "i" icon crash when displaying the leaks details Dec 30, 2018
@jcislinsky
Copy link

Looks like infinite loop.
screenshot 2019-01-07 at 13 14 29

@menesh2
Copy link
Author

menesh2 commented Jan 7, 2019

why is this happening?
is it something that i do?

@jcislinsky
Copy link

How do you initialize LifetimeTracker. In my project I initialize sdk from Obj-c. After initialization from Swift it doesn't crash anymore.

With this initialization it crashes in infinite loop

LifetimeTrackerDashboardIntegration *dashboardIntegration = [LifetimeTrackerDashboardIntegration new];
[LifetimeTracker setupOnUpdate:^(NSDictionary<NSString *,EntriesGroup *> * groups) {
    [dashboardIntegration refreshUIWithTrackedGroups: groups];
}];

With this initialization it doesn't

let dashboardIntegration = LifetimeTrackerDashboardIntegration()
LifetimeTracker.setup { (groups) in
    dashboardIntegration.refreshUI(trackedGroups: groups)
}

@menesh2
Copy link
Author

menesh2 commented Jan 7, 2019

I use this line of code :
LifetimeTracker.setup(onUpdate: LifetimeTrackerDashboardIntegration(visibility: .visibleWithIssuesDetected).refreshUI)

@jcislinsky
Copy link

Another try, it was not dependant on language but when initialization is called. In my case i crashes before window.makeKeyAndVisible().

@menesh2
Copy link
Author

menesh2 commented Jan 7, 2019

I use this line of code :
LifetimeTracker.setup(onUpdate: LifetimeTrackerDashboardIntegration(visibility: .visibleWithIssuesDetected).refreshUI)

what do you think about this initialisation?

Another try, it was not dependant on language but when initialization is called. In my case i crashes before window.makeKeyAndVisible().

i am not calling window.makeKeyAndVisible() on any step

@jcislinsky
Copy link

You can try snooze initialization a little bit. If works for me.

DispatchQueue.main.asyncAfter(deadline: .now() + 1) {
    LifetimeTracker.setup(onUpdate: LifetimeTrackerDashboardIntegration(visibility: .visibleWithIssuesDetected, style: .bar).refreshUI)
}

@menesh2
Copy link
Author

menesh2 commented Jan 7, 2019

You can try snooze initialization a little bit. If works for me.

DispatchQueue.main.asyncAfter(deadline: .now() + 1) {
    LifetimeTracker.setup(onUpdate: LifetimeTrackerDashboardIntegration(visibility: .visibleWithIssuesDetected, style: .bar).refreshUI)
}

Nope, still crashes

@jcislinsky
Copy link

Weird, then I don't know. You must wait for authors to look at crash. You can fork repository and try to simulate crash in Example project (I tried that without success).

@menesh2
Copy link
Author

menesh2 commented Jan 7, 2019

Ok, thanks a lot for trying

@thomasMinshull
Copy link

I'm running into the same issue, did you ever track down what was going on?

@thomasMinshull
Copy link

In my case I'm getting the same error (apparent stack overflow) as a workaround I have configured the lib to use the bubble overlay and not that bar.

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

No branches or pull requests

3 participants