<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div>
> I was back and forth on that issue.  Also, I'm not fully convinced that the<br>
> returning tuples is the right thing to do. It might be better to create a<br>
> class called "Inclusion" that explicitly names the including file, included<br>
> file, location, depth, etc.<br>
<br>
</div>Yeah, I haven't looked closely, but I think it might make sense to have a class.<br></blockquote></div><div><br>Let me rewrite that and post a new patch for consideration.<br></div></div></blockquote></div><br>
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."<br>
<br>I'm not sure about the name of the class or attributes. They make sense to me, but there may be more appropriate names.<br><br>Andrew Sutton<br><a href="mailto:andrew.n.sutton@gmail.com">andrew.n.sutton@gmail.com</a><br>