On 02/06/2013 10:55 AM, Jan Cholasta wrote:
On 5.2.2013 15:45, Petr Viktorin wrote:
On 02/05/2013 01:38 PM, Jan Cholasta wrote:
On 4.2.2013 15:49, Petr Viktorin wrote:
[...]


I see one of the changes is using has_key instead of `in` for a CIDict.
Given that dict.has_key() is deprecated, I think a better solution
would
be to add __contains__ to CIDict. Is there a reason against that?

I think a separate patch with this and other changes to make CIDict more
like dict would be better.

OK, I'll make a patch.

[...]
Updated patches attached.


The changes look good but I can no longer apply the patches. Can you
please rebase them?



Sure.

ACK

--
PetrĀ³

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to