On 05/12/2013 08:19, Argyrios Kyrtzidis wrote:
+typedef struct {
+  /** \brief The number of ranges in the \c ranges array. */
+  unsigned count;
+  /**
+   * \brief An array of \c CXSourceRange, where each range starts at the
+   * preprocessor directive after the # token, and ends at the end of the
+   * corresponding endif.
+   */
+  CXSourceRange *ranges;
+} CXSkippedRanges;
+

CXSourceRangeList or CXRanges would be a more future-proof name for this type than CXSkippedRanges.

It'll be a pain to rename or duplicate the type when another function comes along that also works in terms of sets of source ranges given that this API has a stability guarantee.

Sorry I missed the review thread.

--
http://www.nuanti.com
the browser experts

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to