appsquickly / typhoon-example

An example application built with Typhoon dependency injection framework.
Apache License 2.0
251 stars 59 forks source link

EXC_BAD_ACCESS on initial running of repository #1

Closed 12s closed 10 years ago

12s commented 11 years ago

Hey Jasper, Wanted to try out Typhoon and was attempting to do the demo. Was wondering if you had seen this problem before. I cloned the repo and built the project in Xcode 4.6.1 to my Ipad simulating an iphone or with ios simulator. I get the following error before ever having to enter the world weather API key.

2013-04-03 14:24:05.620 PocketForecast[67184:c07] Property setter invocation: <NSInvocation: 0x6b1f0e0> 2013-04-03 14:24:05.621 PocketForecast[67184:c07] Parsing typeCode: i 2013-04-03 14:24:05.621 PocketForecast[67184:c07] Property setter invocation: <NSInvocation: 0x6b1f5c0> 2013-04-03 14:24:05.651 PocketForecast[67184:c07] -[PFWeatherReportDaoFileSystemImpl filePathFor:][Line 40] [DEBUG] Filepath for archiving: /Users/matt/Library/Application Support/iPhone Simulator/5.1/Applications/D0520CAC-BD7E-461A-84F4-A2604943E812/Documents/weatherReport$$Manila 2013-04-03 14:24:05.655 PocketForecast[67184:c07] -[PFWeatherReportView layoutSubviews][Line 101] [DEBUG] Layout views 2013-04-03 14:24:05.776 PocketForecast[67184:c07] $$$$$$ PFAssembly in dealloc!

Have you seen this before? I noticed in the long file path i don't have anything in the Documents/ folder

Thanks! Matt

jasperblues commented 11 years ago

Hi Matt,

I usually do a few test runs before releasing, including on a fresh install (ie no docs dir) . . . . and didn't come across it. . . However I have an idea what it is. . .

Thanks for the heads-up - will fix today.

Jasper

jasperblues commented 11 years ago

I'm sorry, but I won't actually be able to have a look at this today as promised. Will have some more time in the next few days. . . Do you have an iOS6 simulator installed as well ? Can you try on this. . . (I neglected to test on 5.1).

12s commented 11 years ago

I am getting the error on 6.1 as well. It occurs after I click on a specific city, after your app has loaded.

jasperblues commented 11 years ago

Can you try running the tests and tell me what happens?

(The integration tests require the API key to be set).

lotz commented 11 years ago

Hi all,

This is related to my recent pull request to update the weather feed URL. The old URL was returning an error, and the code calling it was passing in a nil block for the error condition. I'll submit another pull request to fix that up if it happens in the future.

Thanks! Kurt

jasperblues commented 11 years ago

tsk tsk - That was quite silly of me to allow a nil block in there! . . . . thanks for your contribution.