Warning in HCIsInstanceOf

[hamcrest]

Warning in HCIsInstanceOf

[scottrhoyt]

This seems to be because the compiler is looking at the method definition in CAEmitterBehavior.h instead of the definition from the superclass in HCClassMatcher.h. Seeing as HCIsInstanceOf is not a child class of CAEmitterBehavior, I'm not sure why this is. Could be a compiler or Xcode issue?

[herzi]

I'm receiving the same warning in HCIsTypeOf.m as well.

Should I open a ticket at Apple's bug tracker?

[scottrhoyt]

I haven't dug deeper on this to figure out where the problem lies, so I cannot say whether it is an Apple bug or not. Still might be worth opening a ticket. Has anyone seen this error in the newest beta of Xcode?

[jtligon]

can confirm, still present in latest beta.
in addition, the new __nonnull warning is present as well.

HCIsInstanceOf.m:11:39: Incompatible pointer types sending 'Class' to parameter of type 'NSString * __nonnull'

[jonreid]

I didn't try Xcode 7 beta 1, but I don't see it when I build OCHamcrest in beta 2.

[herzi]

This issue is also fixed with the initial Xcode 7 beta. I think we can close this ticket now.

@jtligon Did you Clean the project before trying to build with Xcode 7?

[jtligon]

Clean and build sent it away for Xcode 7 beta 1. thanks!

[jonreid]

Still happens with Xcode 6.4. Oh well, 7 is on the way.

[jonreid]

Fails on beta 4. :(

[herzi]

Just my two cents: To me, this clearly is a bug in Apple's toolchain. We should close this ticket and raise tickets at http://bugreport.apple.com/ to get this solved.

[jonreid]

OK, hold on: I don't see it in the Xcode 7 if I build OCHamcrest independently. I do see it in Xcode 6.4 for when I build the MarvelBrowser project (which pulls in OCHamcrest as a Pod).

Looking more carefully at the signature, it seems to be getting confused by -initWithType: in CAEmitterBehavior.h, which takes an NSString.

[jonreid]

Fixed in d7c0f55 by renaming the method. Apple could do better on this, since [self alloc] returns an instancetype.

[rachten]

@jonreid; Are you planning to officially release a version that incorporates this change?

[jonreid]

Just released v4.2.0

[rachten]

Thanks!