[PATCH] Python/CIndex include file support (take 2)

Clang Frontend
1

I was back and forth on that issue. Also, I’m not fully convinced that the
returning tuples is the right thing to do. It might be better to create a
class called “Inclusion” that explicitly names the including file, included
file, location, depth, etc.

Yeah, I haven’t looked closely, but I think it might make sense to have a class.

Let me rewrite that and post a new patch for consideration.

Here’s a 2nd take on the file inclusions patch. The includes property is now a get_includes function. I’ve added a class FileInclusion that provides a) the source (including) file, the target (included) file, location, depth, and a property for determining if the inclusion is the “input file.”

I’m not sure about the name of the class or attributes. They make sense to me, but there may be more appropriate names.

Andrew Sutton
andrew.n.sutton@gmail.com

py.diff (7.46 KB)

2

Hi Andrew,

Looks great to me, applied in r96106 with a tweak in r96107. Thanks
for the example, too! Now we just need a good viz tool for viewing the
hefty .dot files that come out of clang. :slight_smile:

- Daniel