Include forward class declarations in analysis

Open vladimirnp opened this issue 12 years ago • 3 comments

It would be great if the graph also included forward class declarations as dependencies. Some coding conventions call for using them instead of #import, and having an #import only in the .m file.

Apr 29 '13 13:04 vladimirnp

objc_dep scans also .m files and finds all #import in them. Is this enough?

Jun 19 '13 12:06 jomnius

I think so, but when I used objc_dep 2 months ago I'm pretty sure that didn't work as expected. (I tested it on Facebook's 3.2.1 SDK, and the graph was missing all the #imports that were defined in the .m files only.)

Jun 19 '13 13:06 vladimirnp

Just checked this in a much smaller app, imports are found via .m files. Cannot reproduce, sorry...

However if first header has class forward declaration and actual class import is not in .m file but found via another imported header, then first class has no visible dependency with forward declared class. So yes, looks like a place to improve.

Jun 19 '13 13:06 jomnius