Hello Johannes,

the good thing is that the lib uses LGPL license and it looks quit good.

the bad news is that your code needs some tweaks. First of all the annoying
WS and the please use php_error_docref instead of other error functions.
But most of all never use sprintf. Use s[np]printf. If you are sure
sprintf doesn't cause any trouble recheck it, run it in a debugger and
calculate the bounds (also for 64bit machiens) and reconsider its usage.
If you are still sure sprintf works then comment on it working and why.
But then again you do not need printf att all. Look at php_error_docref.

marcus

Monday, April 26, 2004, 5:06:01 PM, you wrote:

> New idn PECL extension (GNU libidn) as proposed in
> http://marc.theaimsgroup.com/?l=pecl-dev&m=108282209813233&w=2




-- 
Best regards,
 Marcus                            mailto:[EMAIL PROTECTED]

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

Reply via email to