[cfe-commits] [PATCH 2/5] [clang.py] Implement Type.__eq__ and Type.__ne__

Gregory Szorc gregory.szorc at gmail.com
Tue Feb 21 09:22:26 PST 2012


Author: gps
Date: Mon Feb 20 11:44:49 2012
New Revision: 150969

URL: http://llvm.org/viewvc/llvm-project?rev=150969&view=rev
Log:
[clang.py] Implement Type.__eq__ and Type.__ne__



On Mon, Feb 20, 2012 at 8:28 AM, Manuel Klimek <klimek at google.com> wrote:

> Looks good.
>
> On Sat, Feb 18, 2012 at 8:26 PM, Gregory Szorc <gregory.szorc at gmail.com>
> wrote:
> > I've made the implementation of Type.__eq__ more robust and added tests.
> >
> >
> > On Fri, Feb 17, 2012 at 1:37 AM, Manuel Klimek <klimek at google.com>
> wrote:
> >>
> >> On Fri, Feb 17, 2012 at 8:56 AM, Gregory Szorc <gregory.szorc at gmail.com
> >
> >> wrote:
> >> > The implementation could probably be a bit more robust (looking for
> >> > None, comparing types, etc). Do we care?
> >>
> >> Regarding looking for None: what happens if one calls t == None on a
> >> Type? Perhaps add a test?
> >>
> >> Cheers,
> >> /Manuel
> >>
> >>
> >> > ---
> >> >  bindings/python/clang/cindex.py           |   10 ++++++++++
> >> >  bindings/python/tests/cindex/test_type.py |   17 +++++++++++++++++
> >> >  2 files changed, 27 insertions(+), 0 deletions(-)
> >> >
> >> > _______________________________________________
> >> > cfe-commits mailing list
> >> > cfe-commits at cs.uiuc.edu
> >> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >> >
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120221/c5112449/attachment.html>


More information about the cfe-commits mailing list