Re: [Freeipa-devel] [PATCHES] 0177-0179 Add missing dict methods to CIDict

2013-09-24 Thread Jan Cholasta
On 23.9.2013 12:08, Petr Viktorin wrote: On 09/23/2013 09:18 AM, Jan Cholasta wrote: On 18.9.2013 14:00, Petr Viktorin wrote: On 09/17/2013 05:13 PM, Jan Cholasta wrote: On 20.2.2013 17:37, Petr Viktorin wrote: On 02/19/2013 01:51 PM, Jan Cholasta wrote: Hi, On 5.2.2013 18:02, Petr

Re: [Freeipa-devel] [PATCHES] 0177-0179 Add missing dict methods to CIDict

2013-09-23 Thread Jan Cholasta
On 18.9.2013 14:00, Petr Viktorin wrote: On 09/17/2013 05:13 PM, Jan Cholasta wrote: On 20.2.2013 17:37, Petr Viktorin wrote: On 02/19/2013 01:51 PM, Jan Cholasta wrote: Hi, On 5.2.2013 18:02, Petr Viktorin wrote: CIDict, our case-insensitive dictionary, inherits from dict but did not

Re: [Freeipa-devel] [PATCHES] 0177-0179 Add missing dict methods to CIDict

2013-09-23 Thread Petr Viktorin
On 09/23/2013 09:18 AM, Jan Cholasta wrote: On 18.9.2013 14:00, Petr Viktorin wrote: On 09/17/2013 05:13 PM, Jan Cholasta wrote: On 20.2.2013 17:37, Petr Viktorin wrote: On 02/19/2013 01:51 PM, Jan Cholasta wrote: Hi, On 5.2.2013 18:02, Petr Viktorin wrote: CIDict, our case-insensitive

Re: [Freeipa-devel] [PATCHES] 0177-0179 Add missing dict methods to CIDict

2013-09-17 Thread Jan Cholasta
On 20.2.2013 17:37, Petr Viktorin wrote: On 02/19/2013 01:51 PM, Jan Cholasta wrote: Hi, On 5.2.2013 18:02, Petr Viktorin wrote: CIDict, our case-insensitive dictionary, inherits from dict but did not reimplement the full dict interface. Calling the missing methods silently invoked

Re: [Freeipa-devel] [PATCHES] 0177-0179 Add missing dict methods to CIDict

2013-02-20 Thread Petr Viktorin
On 02/19/2013 01:51 PM, Jan Cholasta wrote: Hi, On 5.2.2013 18:02, Petr Viktorin wrote: CIDict, our case-insensitive dictionary, inherits from dict but did not reimplement the full dict interface. Calling the missing methods silently invoked case-sensitive behavior. Our code seems to avoid

Re: [Freeipa-devel] [PATCHES] 0177-0179 Add missing dict methods to CIDict

2013-02-19 Thread Jan Cholasta
Hi, On 5.2.2013 18:02, Petr Viktorin wrote: CIDict, our case-insensitive dictionary, inherits from dict but did not reimplement the full dict interface. Calling the missing methods silently invoked case-sensitive behavior. Our code seems to avoid that, but it's a bit of a minefield for new

[Freeipa-devel] [PATCHES] 0177-0179 Add missing dict methods to CIDict

2013-02-05 Thread Petr Viktorin
CIDict, our case-insensitive dictionary, inherits from dict but did not reimplement the full dict interface. Calling the missing methods silently invoked case-sensitive behavior. Our code seems to avoid that, but it's a bit of a minefield for new development. Patch 119 adds the missing dict