Re: Re: [PHP4BETA] cvs: /php4/ext/imap imap.c

> Quoting Rasmus Lerdorf <rasmus@php.net>: > > > if ( body->id ){ > > add_property_long( return_value, "ifid", 1 ); > > - if(body->description) add_property_string( return_value, "id", > > body->description, 1 ); > > + if(body->description) add_property_string( return_value, "id", > > body->id, 1 ); > > } else { > > add_property_long( return_value, "ifid", 0 ); > > } > > I'm a bit confused about the logic here. Why should body->description be > checked at all - if body->id is there, it should be used. Or am I missing > something? No idea. body->description is checked all over. Who wrote this function? Mark perhaps? I think we need some insight. -Rasmus