Date: 10/30/99
- Next message: Zeev Suraski: "[PHP-DEV] New compile failure"
- Previous message: Rasmus Lerdorf: "Re: [PHP-DEV] Un*x Security Experts"
- In reply to: Chuck Hagenbuch: "[PHP-DEV] Re: [PHP4BETA] cvs: /php4/ext/imap imap.c"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
> Quoting Rasmus Lerdorf <rasmus <email protected>>:
>
> > 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
-- PHP Development Mailing List <http://www.php.net/> To unsubscribe, e-mail: php-dev-unsubscribe <email protected> For additional commands, e-mail: php-dev-help <email protected> To contact the list administrators, e-mail: php-list-admin <email protected>
- Next message: Zeev Suraski: "[PHP-DEV] New compile failure"
- Previous message: Rasmus Lerdorf: "Re: [PHP-DEV] Un*x Security Experts"
- In reply to: Chuck Hagenbuch: "[PHP-DEV] Re: [PHP4BETA] cvs: /php4/ext/imap imap.c"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

