Skip to content

Crashes on ImageFilter.m #215

Closed
Groxx opened this Issue May 6, 2012 · 2 comments

2 participants

@Groxx
Groxx commented May 6, 2012

I've been trying to figure out what the cause is, but I think I'm wading in too deep.
This file: https://github.com/esilverberg/ios-image-filters/blob/master/Classes/ImageFilter.m
is causing a crash while parsing: *** Assertion failure in +[GBMethodData methodDataWithType:result:arguments:], /<dir>/appledoc/Model/GBMethodData.m:39
and I'm not sure what other context I can provide. Running it in isolation with --no-create-html crashes, let me know if I can help more!

@tomaz
Owner
tomaz commented May 7, 2012

It's because C functions inside category implementation - C functions are not supported by current version (see #2). If I delete C functions, it works.

Although this is valid syntax regarding compiler, it's probably not the best approach to mix two interfaces IMHO. You will probably be better off by moving C interface out of Objective C categories or classes. Perhaps even to separate file to make them more readable. At least for me it starts ringing alarms after file gets over couple hundred lines, but that's just my opinion :)

BTW: I'm now working on next version which will support more features. It currently accepts enumerations and structs, but not yet C functions. I tried running it over your file and it extracts the three structs, but not the category. Didn't check why though. You can follow progress on Twitter @gentlebytes or blog. I've also written blog post explaining the motive and directions for future version. You can also follow along the coding - it's on experimental branch.

I'm closing this issue, feel free to bump a comment on #2 to be notified of progress.

@tomaz tomaz closed this May 7, 2012
@Groxx
Groxx commented May 7, 2012

Sounds great, and I'll check out the post / progress. Thanks, and impressive speed on the reply! It's always nice to see an active project maintainer.

For the time being, since I don't really care about that file, I've just excluded it from the output. There's no real documentation in the file anyway (as seems to be normal for iOS mini-libraries... argh).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.