Jdmoreira master

[onevcat]

No description provided.

[codecov-io]

Current coverage is 72.55% (diff: 68.39%)

Merging #430 into master will decrease coverage by 1.72%

@@             master       #430   diff @@
==========================================
  Files            14         16     +2   
  Lines          1691       1767    +76   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           1256       1282    +26   
- Misses          435        485    +50   
  Partials          0          0          

Powered by Codecov. Last update b1fa34e...841c6b6

[jdmoreira]

I've seen you merged and did a few changes. I'm sorry I didn't had much time the last couple of days. Thanks for taking the time.

I agree that the keeping the objc_set/getObject and using Box is a much better solution 👍

[onevcat]

@jdmoreira Hi, I think your wrapper solution is also great. But the blocking is that it cannot compile in Release mode with -O optimization. It crashes LLVM in the conversion of T to AnyObject. Furthermore, in Swift 3 we cannot depend on AnyObject to check class (since everything is imported as Any now). So I guess it would be better to leave it to us developers to decide which should be in box.

I am working on Swift 3 for the indicator now. Glad and honor that you help to improve it!