4. It's not conditional and who says everthing likes it that
    way just becayse you want it that way? Hmm ...

    My opinion is this patch/method shouldn't be considered at
    all. First discuss implementation issues better...

On Sat, Mar 30, 2002 at 12:56:11PM +0100, [EMAIL PROTECTED] wrote : 
> Hello,
> 
> 1. this patch doesn't work with thread safety, 2. It doesn't use the PHP 
> coding standards and 3. it uses GOTO. I don't think this should have been 
> committed at all.
> 
> Derick
> 
> 
> 
> On Sat, 30 Mar 2002, Jan Pavlík wrote:
> 
> > rootshell  Fri Mar 29 23:13:29 2002 CET
> > 
> >   Modified files:
> >      /php4/main/main.c
> >   Log:
> >     lot of same errors are logged as one, cleans errorlog
> > 
> > 
> > 
> > Index: php4/main/main.c
> > diff -u php4/main/main.c:1.404.2.5 php4/main/main.c:1.404.2.6
> > --- php4/main/main.c:1.404.2.5 Mon Dec 17 22:19:51 2001
> > +++ php4/main/main.c Fri Mar 29 23:11:03 2002
> > @@ -18,11 +18,12 @@
> >     +----------------------------------------------------------------------+
> >  */
> > 
> > -/* $Id: main.c,v 1.404.2.5 2001/12/17 21:19:51 derick Exp $ */
> > +/* $Id: main.c,v 1.404.2.6 2002/03/29 23:10:05 rootshell Exp $ */
> > 
> >  /* {{{ includes
> >   */
> >  #include <stdio.h>
> > +#include <string.h>
> >  #include "php.h"
> >  #ifdef PHP_WIN32
> >  #include "win32/time.h"
> > @@ -83,6 +84,9 @@
> >  PHPAPI int core_globals_id;
> >  #endif
> > 
> > +static char _err_buf[1024];
> > +#define MAX_ERR_LENGTH (buffer_len > sizeof(_err_buf) ? sizeof(_err_buf) :
> > buffer_len)
> > +
> >  static void php_build_argv(char *s, zval *track_vars_array TSRMLS_DC);
> > 
> > 
> > @@ -417,6 +421,9 @@
> >     char *append_string = INI_STR("error_append_string");
> >     char *error_format;
> > 
> > +   if(!strncmp(buffer, _err_buf, MAX_ERR_LENGTH)) goto _away;
> > +   else strncpy(_err_buf, buffer, MAX_ERR_LENGTH);
> > +
> >     error_format = PG(html_errors) ?
> >      "<br>\n<b>%s</b>:  %s in <b>%s</b> on line <b>%d</b><br>\n"
> >      : "\n%s: %s in %s on line %d\n";
> > @@ -437,6 +444,7 @@
> >     if (append_string) {
> >      PUTS(append_string);
> >     }
> > +_away:
> >    }
> >  #if ZEND_DEBUG
> >    {
> > 
> > 
> > 
> > 
> > 
> > 
> > 
> > -- 
> > PHP Development Mailing List <http://www.php.net/>
> > To unsubscribe, visit: http://www.php.net/unsub.php
> > 
> 
> -----------------------------------------------------------------------
>               PHP: Scripting the Web - [EMAIL PROTECTED]
>                 All your branches are belong to me!
>             SRM: Script Running Machine - www.vl-srm.net
> -----------------------------------------------------------------------
> 
> 
> -- 
> PHP Development Mailing List <http://www.php.net/>
> To unsubscribe, visit: http://www.php.net/unsub.php

-- 
Please always Cc to me when replying to me on the lists.
GnuPG Key: http://guru.josefine.at/~mfischer/C2272BD0.asc
"Mind if I MFH ?" "What QA did you do on it?" "the usual?" "ah... none :)"

-- 
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to