Skip to content
This repository has been archived by the owner on Feb 8, 2022. It is now read-only.

Fix incorrect thread access #56

Merged
merged 1 commit into from Jan 6, 2016

Conversation

ngs
Copy link
Contributor

@ngs ngs commented Jan 5, 2016

These method calls are causing IncorrectThreadException

libc++abi.dylib: terminating with uncaught exception of type
realm::IncorrectThreadException: Realm accessed from incorrect thread.

@polqf
Copy link
Owner

polqf commented Jan 5, 2016

Hi, @ngs , could you explain in which case are you getting this error? Xcode version, RRC version, install method...?

We are using RRC in a huge App and this has never happened to us.

A chuck of code from where the error is generated would be highly appreciated

@ngs
Copy link
Contributor Author

ngs commented Jan 6, 2016

I saw this crash both simulator and actual iPhone device.

Here is the crash log: https://gist.github.com/ngs/84be87109cbe575e27c2

And here is my code: https://github.com/ngs/ci2go/blob/c7ef3b86ccefdb8e27df626916591e8107635f82/CI2Go/Networking/BuildNetworkingExtension.swift#L109

@ngs ngs force-pushed the feature/realm0.97.0 branch 2 times, most recently from abf4ba5 to 3212950 Compare January 6, 2016 06:51
@polqf
Copy link
Owner

polqf commented Jan 6, 2016

Hi @ngs,

We've seen that you are using a predicate that is accessing to relationships, and that's the cause of the crash, for now RRC does not support them.

It seems that this PR is going to fix that, so thanks a lot.
Could you clean the additions and deletions on empty lines please?

@ngs ngs force-pushed the feature/realm0.97.0 branch 3 times, most recently from d0e7aa1 to 5bc6e59 Compare January 6, 2016 15:38
These method calls are causing `IncorrectThreadException`

```
libc++abi.dylib: terminating with uncaught exception of type
realm::IncorrectThreadException: Realm accessed from incorrect thread.
```
@ngs
Copy link
Contributor Author

ngs commented Jan 6, 2016

@poolqf

Thanks for checking the crash log. I've cleaned up the code.

@polqf
Copy link
Owner

polqf commented Jan 6, 2016

Merging.. Thanks @ngs

polqf added a commit that referenced this pull request Jan 6, 2016
@polqf polqf merged commit 1b3d54a into polqf:feature/realm0.97.0 Jan 6, 2016
@polqf polqf modified the milestone: 0.4.0 Jan 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants