<div dir="ltr">But won't you have to change this code if you change the docs to say "returns zero if the set contains the specified cursor"<div><br></div><div><div> CXCursorSet_Impl *setImpl = unpackCXCursorSet(set);</div>

<div>  if (!setImpl)</div><div>    return 0;</div></div><div><br></div><div>???</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 24, 2013 at 5:20 PM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com" target="_blank">kremenek@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">The comments in the PR were to keep the function, so I won't rip it out.<div><br>

</div><div>That said, if there are clients, I'd prefer we change the documentation, not the behavior of the code itself.</div><div><br><div><div><div class="h5"><div>On Apr 23, 2013, at 11:48 PM, Ted Kremenek <<a href="mailto:kremenek@apple.com" target="_blank">kremenek@apple.com</a>> wrote:</div>

<br></div></div><blockquote type="cite"><div><div class="h5"><div style="word-wrap:break-word">Thanks Nikola.  This PR and patch made me go back and re-evaluate the purpose and need for this function.  I don't think there are any clients of CXCursorSet, so I think I will just remove it entirely.<div>

<br></div><div>Thanks for investigating this.<br><div><br><div><div>On Apr 22, 2013, at 2:12 AM, Nikola Smiljanic <<a href="mailto:popizdeh@gmail.com" target="_blank">popizdeh@gmail.com</a>> wrote:</div><br><blockquote type="cite">

<div dir="ltr">I was going through bugizilla when I ran into this one <a href="http://llvm.org/bugs/show_bug.cgi?id=10124" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=10124</a>  Bug and patch look good but I wanted to double check if this is OK to commit and close?<div>



<br></div><div>CCing Ted since he seems to be the author of the function.</div>
</div>
</blockquote></div><br></div></div></div></div></div>
_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>

</blockquote></div><br></div></div></blockquote></div><br></div>