[Code-review] review: Gadget act-props branch

Guillaume Desmottes guillaume.desmottes at collabora.co.uk
Wed Aug 13 09:47:27 EDT 2008


Le mercredi 13 août 2008 à 12:42 +0100, Dafydd Harries a écrit :
> > > > > >    for child in node.elements():
> > > > > >+        if child.uri == ns.ACTIVITY_PROP and child.name == 'properties':
> > > > > >+            for child in child.elements():
> > > > > >+                assert child.name == 'property'
> > > > > 
> > > > > I think it's better to just ignore unexpected elements:
> > > > > 
> > > > >   for child in child.elements():
> > > > >       if child.name != 'property':
> > > > >           continue
> 
> Oops: I didn't realise that this was in the test code. I don't think it
> matters there.


I already changed it so...

OK, to merge act-props in its current state?



	G.




More information about the Code-review mailing list