Re: [PHP-DEV] Re: crc32() improvements

2004-06-14 Thread Mike Benoit
On Mon, 2004-06-14 at 12:48 +0200, Christian Schneider wrote: > Alexander Valyalkin wrote: > > It is only idle talk. Can you provide any string from my code which > > violates your "coding standards"? > > Calm down. As I said before (obviously not clearly enough, I was hoping > one of the 'projec

Re: [PHP-DEV] Re: crc32() improvements

2004-06-14 Thread Alexander Valyalkin
On Mon, 14 Jun 2004 13:49:16 +0200, Ard Biesheuvel <[EMAIL PROTECTED]> wrote: Alexander Valyalkin wrote: :) Are you sure? I'm not. Look on declaration of [nr] variable: int nr; And answer, please, which value will be assigned to nr, if length of a string will be greater than 0x7fff on 32-bit ar

Re: [PHP-DEV] Re: crc32() improvements

2004-06-14 Thread Ard Biesheuvel
Alexander Valyalkin wrote: :) Are you sure? I'm not. Look on declaration of [nr] variable: int nr; And answer, please, which value will be assigned to nr, if length of a string will be greater than 0x7fff on 32-bit architecture? The funny thing is that in this case, it doesn't matter if 'nr' is

Re: [PHP-DEV] Re: crc32() improvements

2004-06-14 Thread Christian Schneider
Alexander Valyalkin wrote: It is only idle talk. Can you provide any string from my code which violates your "coding standards"? Calm down. As I said before (obviously not clearly enough, I was hoping one of the 'project managers' would do that for me ;-)) you are missing the point why people rej

Re: [PHP-DEV] Re: crc32() improvements

2004-06-14 Thread Alexander Valyalkin
On Mon, 14 Jun 2004 11:00:33 +0200 (CEST), Derick Rethans <[EMAIL PROTECTED]> wrote: if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &p, &nr) == FAILURE) { return; } /* !!! there is no error check nr < 0 */ Of course not, that's pointless as a string can never

Re: [PHP-DEV] Re: crc32() improvements

2004-06-14 Thread Derick Rethans
On Mon, 14 Jun 2004, Alexander Valyalkin wrote: > Thank you for good explanation and comments. > Now I understood that the current crc32 implementation is better than mine. > But it consists some ugly bugs (read my comments): > PHP_NAMED_FUNCTION(php_if_crc32) > { > unsigned int crc = ~0; >