Re: [Freeipa-devel] [PATCH 0105] Remove dead assignment in ldap_parse_zoneentry()

2012-11-23 Thread Petr Spacek
On 11/22/2012 02:33 PM, Adam Tkac wrote: On Thu, Nov 22, 2012 at 12:30:00PM +0100, Petr Spacek wrote: >Hello, > >this patch fixes the last warning from Clang. Any new warning should >attract our attention. > >Commit message: > Remove dead assignment in ldap_parse_zoneentry(). > > Assigne

Re: [Freeipa-devel] [PATCH 0105] Remove dead assignment in ldap_parse_zoneentry()

2012-11-22 Thread Adam Tkac
On Thu, Nov 22, 2012 at 12:30:00PM +0100, Petr Spacek wrote: > Hello, > > this patch fixes the last warning from Clang. Any new warning should > attract our attention. > > Commit message: > Remove dead assignment in ldap_parse_zoneentry(). > > Assigned result value was never read because

[Freeipa-devel] [PATCH 0105] Remove dead assignment in ldap_parse_zoneentry()

2012-11-22 Thread Petr Spacek
Hello, this patch fixes the last warning from Clang. Any new warning should attract our attention. Commit message: Remove dead assignment in ldap_parse_zoneentry(). Assigned result value was never read because it is overwritten by following CHECK()s in any case. Dead assignme