Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Zeev Suraski

snprintf() isn't accessible in the Zend engine...

Zeev

At 08:43 12/3/2001, Sebastian Bergmann wrote:
Andrei Zmievski wrote:
  andrei  Mon Mar 12 05:08:29 2001 EDT
 
Modified files:
  /Zend   zend_API.c zend_API.h
Log:
Improve zend_is_callable() to the point where it's actually useful.
Now it just needs to be invoked everywhere in PHP where a callback is
expected.

   This breaks the Win32 build:

zend_API.c
D:\Programme\MS Visual Studio\Projekte\php\php4\Zend\zend_API.c(958) :
warning C4013: 'snprintf' undefiniert; Annahme: extern mit Rueckgabetyp int

ZendTS.lib(zend_API.obj) : error LNK2001: Nichtaufgeloestes externes Symbol
_snprintf
..\Release_TS_inline\php4ts.dll : fatal error LNK1120: 1 unaufgeloeste
externe Verweise

--
  sebastian bergmann e-mail :  [EMAIL PROTECTED]
   homepage :  http://www.sebastian-bergmann.de
make a gift : http://wishlist.sebastian-bergmann.de
  measure the usability of your web application - http://phpOpenTracker.de

--
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]

--
Zeev Suraski [EMAIL PROTECTED]
CTO   co-founder, Zend Technologies Ltd. http://www.zend.com/


-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Andrei Zmievski

Use zend_snprintf() instead or what?

On Mon, 12 Mar 2001, Zeev Suraski wrote:
 snprintf() isn't accessible in the Zend engine...
 
 Zeev
 
 At 08:43 12/3/2001, Sebastian Bergmann wrote:
 Andrei Zmievski wrote:
   andrei  Mon Mar 12 05:08:29 2001 EDT
  
 Modified files:
   /Zend   zend_API.c zend_API.h
 Log:
 Improve zend_is_callable() to the point where it's actually useful.
 Now it just needs to be invoked everywhere in PHP where a callback is
 expected.
 
This breaks the Win32 build:
 
 zend_API.c
 D:\Programme\MS Visual Studio\Projekte\php\php4\Zend\zend_API.c(958) :
 warning C4013: 'snprintf' undefiniert; Annahme: extern mit Rueckgabetyp int
 
 ZendTS.lib(zend_API.obj) : error LNK2001: Nichtaufgeloestes externes Symbol
 _snprintf
 ..\Release_TS_inline\php4ts.dll : fatal error LNK1120: 1 unaufgeloeste
 externe Verweise
 
 --
   sebastian bergmann e-mail :  [EMAIL PROTECTED]
homepage :  http://www.sebastian-bergmann.de
 make a gift : http://wishlist.sebastian-bergmann.de
   measure the usability of your web application - http://phpOpenTracker.de
 
 --
 PHP Development Mailing List http://www.php.net/
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 To contact the list administrators, e-mail: [EMAIL PROTECTED]
 
 --
 Zeev Suraski [EMAIL PROTECTED]
 CTO   co-founder, Zend Technologies Ltd. http://www.zend.com/
 



-Andrei

Any sufficiently advanced bug is
indistinguishable from a feature.
-- Rich Kulawiec

-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Zeev Suraski

At 16:09 12/3/2001, Andrei Zmievski wrote:
Use zend_snprintf() instead or what?

No, it's really inaccessible :)  We managed to write the engine without 
needing it...  Is it impossible to replace it with other functions?

Zeev


-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Andrei Zmievski

On Mon, 12 Mar 2001, Jason Greene wrote:
 strlcpy, and strlcat are in the win32 build (main/strlcat.c main/strlcpy.c)
 why dont you malloc a buffer that would fit the sprintf data, and then strlcpy
 the size restriction?

Hmm, and how would you know how big the sprintf buffer would need to be
for an arbitrary format and arguments? :) In my case it's much simpler,
so yes, I could do that, but otherwise.. That's why Glib has
g_strdup_printf() function.. quite nice.

-Andrei
* If it ain't broken, it doesn't have enough features yet. *

-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Jason Greene

Simple, allocate an arbitrary large number, like a gig, give there OS's paging 
routines a test : )
That is a true point, I was thinking that in most cases you could accurately predict 
the size.
g_strdup_printf - that sounds like a fun function, why don't most libraries have this 
feature : )

-Jason



- Original Message - 
From: "Andrei Zmievski" [EMAIL PROTECTED]
To: "Jason Greene" [EMAIL PROTECTED]
Cc: "Zeev Suraski" [EMAIL PROTECTED]; "php-dev mailinglist" [EMAIL PROTECTED]
Sent: Monday, March 12, 2001 10:15 AM
Subject: Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h


 On Mon, 12 Mar 2001, Jason Greene wrote:
  strlcpy, and strlcat are in the win32 build (main/strlcat.c main/strlcpy.c)
  why dont you malloc a buffer that would fit the sprintf data, and then strlcpy
  the size restriction?
 
 Hmm, and how would you know how big the sprintf buffer would need to be
 for an arbitrary format and arguments? :) In my case it's much simpler,
 so yes, I could do that, but otherwise.. That's why Glib has
 g_strdup_printf() function.. quite nice.
 
 -Andrei
 * If it ain't broken, it doesn't have enough features yet. *
 
 -- 
 PHP Development Mailing List http://www.php.net/
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 To contact the list administrators, e-mail: [EMAIL PROTECTED]
 


-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Andi Gutmans

I commited a patch but didn't test it.
Andrei, please make sure I don't have some dumb bug there.

Andi

At 10:12 AM 3/12/2001 -0600, Jason Greene wrote:
strlcpy, and strlcat are in the win32 build (main/strlcat.c main/strlcpy.c)
why dont you malloc a buffer that would fit the sprintf data, and then strlcpy
the size restriction?

-Jason

- Original Message -
From: "Andrei Zmievski" [EMAIL PROTECTED]
To: "Zeev Suraski" [EMAIL PROTECTED]
Cc: "php-dev mailinglist" [EMAIL PROTECTED]
Sent: Monday, March 12, 2001 9:09 AM
Subject: Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h


  On Mon, 12 Mar 2001, Zeev Suraski wrote:
   No, it's really inaccessible :)  We managed to write the engine without
   needing it...  Is it impossible to replace it with other functions?
 
  How do you sprintf to buffer of fixed size without possible overflow? I
  suppose I could use strlcat() to build the string piece by piece. Is
  that available in the engine?
 
  -Andrei
 
  When we eliminate the impossible, whatever remains,
  however improbable, must be true. -- Sherlock Holmes
 
  --
  PHP Development Mailing List http://www.php.net/
  To unsubscribe, e-mail: [EMAIL PROTECTED]
  For additional commands, e-mail: [EMAIL PROTECTED]
  To contact the list administrators, e-mail: [EMAIL PROTECTED]
 


--
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]


-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Andrei Zmievski

On Mon, 12 Mar 2001, Andi Gutmans wrote:
 I commited a patch but didn't test it.
 Andrei, please make sure I don't have some dumb bug there.

Thanks.

-Andrei
* Life may be expensive, but it includes an annual free trip around the sun. *

-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Shane Caraveo

Why not add a:

#define snprintf _snprintf

to the zend_config.w32.h file?  That should take care of the mentioned
compile problem under windows.

- Original Message -
From: "Andrei Zmievski" [EMAIL PROTECTED]
To: "Zeev Suraski" [EMAIL PROTECTED]
Cc: "php-dev mailinglist" [EMAIL PROTECTED]
Sent: Monday, March 12, 2001 6:09 AM
Subject: Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h


 Use zend_snprintf() instead or what?

 On Mon, 12 Mar 2001, Zeev Suraski wrote:
  snprintf() isn't accessible in the Zend engine...
 
  Zeev
 
  At 08:43 12/3/2001, Sebastian Bergmann wrote:
  Andrei Zmievski wrote:
andrei  Mon Mar 12 05:08:29 2001 EDT
   
  Modified files:
/Zend   zend_API.c zend_API.h
  Log:
  Improve zend_is_callable() to the point where it's actually
useful.
  Now it just needs to be invoked everywhere in PHP where a callback
is
  expected.
  
 This breaks the Win32 build:
  
  zend_API.c
  D:\Programme\MS Visual Studio\Projekte\php\php4\Zend\zend_API.c(958) :
  warning C4013: 'snprintf' undefiniert; Annahme: extern mit Rueckgabetyp
int
  
  ZendTS.lib(zend_API.obj) : error LNK2001: Nichtaufgeloestes externes
Symbol
  _snprintf
  ..\Release_TS_inline\php4ts.dll : fatal error LNK1120: 1 unaufgeloeste
  externe Verweise
  
  --
sebastian bergmann e-mail :
[EMAIL PROTECTED]
 homepage :
http://www.sebastian-bergmann.de
  make a gift :
http://wishlist.sebastian-bergmann.de
measure the usability of your web application -
http://phpOpenTracker.de
  
  --
  PHP Development Mailing List http://www.php.net/
  To unsubscribe, e-mail: [EMAIL PROTECTED]
  For additional commands, e-mail: [EMAIL PROTECTED]
  To contact the list administrators, e-mail:
[EMAIL PROTECTED]
 
  --
  Zeev Suraski [EMAIL PROTECTED]
  CTO   co-founder, Zend Technologies Ltd. http://www.zend.com/
 



 -Andrei

 Any sufficiently advanced bug is
 indistinguishable from a feature.
 -- Rich Kulawiec

 --
 PHP Development Mailing List http://www.php.net/
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 To contact the list administrators, e-mail: [EMAIL PROTECTED]




-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-12 Thread Andi Gutmans

Because snprintf() sucks and what we did is faster :)

Andi

At 09:25 AM 3/12/2001 -0800, Shane Caraveo wrote:
Why not add a:

#define snprintf _snprintf

to the zend_config.w32.h file?  That should take care of the mentioned
compile problem under windows.

- Original Message -
From: "Andrei Zmievski" [EMAIL PROTECTED]
To: "Zeev Suraski" [EMAIL PROTECTED]
Cc: "php-dev mailinglist" [EMAIL PROTECTED]
Sent: Monday, March 12, 2001 6:09 AM
Subject: Re: [PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h


  Use zend_snprintf() instead or what?
 
  On Mon, 12 Mar 2001, Zeev Suraski wrote:
   snprintf() isn't accessible in the Zend engine...
  
   Zeev
  
   At 08:43 12/3/2001, Sebastian Bergmann wrote:
   Andrei Zmievski wrote:
 andrei  Mon Mar 12 05:08:29 2001 EDT

   Modified files:
 /Zend   zend_API.c zend_API.h
   Log:
   Improve zend_is_callable() to the point where it's actually
useful.
   Now it just needs to be invoked everywhere in PHP where a callback
is
   expected.
   
  This breaks the Win32 build:
   
   zend_API.c
   D:\Programme\MS Visual Studio\Projekte\php\php4\Zend\zend_API.c(958) :
   warning C4013: 'snprintf' undefiniert; Annahme: extern mit Rueckgabetyp
int
   
   ZendTS.lib(zend_API.obj) : error LNK2001: Nichtaufgeloestes externes
Symbol
   _snprintf
   ..\Release_TS_inline\php4ts.dll : fatal error LNK1120: 1 unaufgeloeste
   externe Verweise
   
   --
 sebastian bergmann e-mail :
[EMAIL PROTECTED]
  homepage :
http://www.sebastian-bergmann.de
   make a gift :
http://wishlist.sebastian-bergmann.de
 measure the usability of your web application -
http://phpOpenTracker.de
   
   --
   PHP Development Mailing List http://www.php.net/
   To unsubscribe, e-mail: [EMAIL PROTECTED]
   For additional commands, e-mail: [EMAIL PROTECTED]
   To contact the list administrators, e-mail:
[EMAIL PROTECTED]
  
   --
   Zeev Suraski [EMAIL PROTECTED]
   CTO   co-founder, Zend Technologies Ltd. http://www.zend.com/
  
 
 
 
  -Andrei
 
  Any sufficiently advanced bug is
  indistinguishable from a feature.
  -- Rich Kulawiec
 
  --
  PHP Development Mailing List http://www.php.net/
  To unsubscribe, e-mail: [EMAIL PROTECTED]
  For additional commands, e-mail: [EMAIL PROTECTED]
  To contact the list administrators, e-mail: [EMAIL PROTECTED]
 
 


--
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]


-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]




[PHP-DEV] Re: [ZEND-ENGINE] cvs: Zend / zend_API.c zend_API.h

2001-03-11 Thread Sebastian Bergmann

Andrei Zmievski wrote:
 andrei  Mon Mar 12 05:08:29 2001 EDT
 
   Modified files:
 /Zend   zend_API.c zend_API.h
   Log:
   Improve zend_is_callable() to the point where it's actually useful.
   Now it just needs to be invoked everywhere in PHP where a callback is
   expected.

  This breaks the Win32 build:

zend_API.c
D:\Programme\MS Visual Studio\Projekte\php\php4\Zend\zend_API.c(958) :
warning C4013: 'snprintf' undefiniert; Annahme: extern mit Rueckgabetyp int

ZendTS.lib(zend_API.obj) : error LNK2001: Nichtaufgeloestes externes Symbol
_snprintf
..\Release_TS_inline\php4ts.dll : fatal error LNK1120: 1 unaufgeloeste
externe Verweise

-- 
 sebastian bergmann e-mail :  [EMAIL PROTECTED]
  homepage :  http://www.sebastian-bergmann.de
   make a gift : http://wishlist.sebastian-bergmann.de
 measure the usability of your web application - http://phpOpenTracker.de

-- 
PHP Development Mailing List http://www.php.net/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]