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

replicateUITextViewNSTextStorage crash. #280

Open
axenovV opened this issue Dec 28, 2016 · 5 comments
Open

replicateUITextViewNSTextStorage crash. #280

axenovV opened this issue Dec 28, 2016 · 5 comments

Comments

@axenovV
Copy link

axenovV commented Dec 28, 2016

Hello. Sometime I have an crash in this method. I do not get to call her for testing, but it appears people are quite often. This is stacktrace from Crashlytics.
Xcode 8.2, swift 2.2
2016-12-28 17 57 26

@diegosanchezr
Copy link
Contributor

I haven't seen this one, so this could come from some other unrelated code. If you have custom cells double check if they're being used in the background. If nothing works, just try disabling background sizing.

@axenovV
Copy link
Author

axenovV commented Dec 28, 2016

@diegosanchezr Thanks. I will check this and let you know. But my method that crash happened don't have any differences from your implementation.

@axenovV
Copy link
Author

axenovV commented Dec 29, 2016

@diegosanchezr When i change canCalculateSizeInBackground to false, сrash will no longer occur.
But there is another crash associated with Observable in ChattoAdditions:
2016-12-29 15 37 32
This crash occurs periodically when calling PhotoMessageCollectionViewCell.performBatchUpdates. As I understand this is due to the fact that not Observable not thread safe?

@diegosanchezr
Copy link
Contributor

It looks like you have some thread safety issue. Yes, observable is not thread safe, so be careful on how you use it - but the race could also happen in some other unrelated code and manifest there. Thread sanitizer might be useful here, but you would need Swift 3 IIRC. Unfortunately, I don't think I can help you more without a reproducer.

@axenovV
Copy link
Author

axenovV commented Dec 29, 2016

@diegosanchezr Thanks, Then I will say later what the problem is, when correct and translate the project on swift 3.

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

2 participants