Re: [PHP-CVS] cvs: php-src /ext/tidy tidy.c

2006-11-01 Thread Nuno Lopes
In fact they are all the same. When you pass an html string, it opens a new converter (with ref_count=1), and then it just points the child nodes to that converter (and incs the ref_counter). So I'm only storing a pointer to the converter with a ref. counter associated. This is needed because yo

Re: [PHP-CVS] cvs: php-src /ext/tidy tidy.c

2006-11-01 Thread Andrei Zmievski
Do you need a separate converter per node? Can they all be different for some reason? -Andrei On Nov 1, 2006, at 10:28 AM, Nuno Lopes wrote: nlopess Wed Nov 1 18:28:37 2006 UTC Modified files: /php-src/ext/tidy tidy.c Log: start converting the tidy and tidyNode objects t

Re: [PHP-CVS] cvs: php-src /ext/tidy tidy.c

2006-04-23 Thread Nuno Lopes
Yep it will always contain ASCII characters. Thanks for your comment, Nuno - Original Message - Looks fine if you're sure that the release date string will always contain only ASCII characters. -Andrei On Apr 23, 2006, at 5:56 AM, Nuno Lopes wrote: -/* {{{ proto string tidy_get_re

Re: [PHP-CVS] cvs: php-src /ext/tidy tidy.c

2006-04-23 Thread Andrei Zmievski
Looks fine if you're sure that the release date string will always contain only ASCII characters. -Andrei On Apr 23, 2006, at 5:56 AM, Nuno Lopes wrote: -/* {{{ proto string tidy_get_release() +/* {{{ proto string tidy_get_release() U Get release date (version) for Tidy library */ PHP_F

Re: [PHP-CVS] cvs: php-src /ext/tidy tidy.c

2004-04-16 Thread Derick Rethans
On Fri, 16 Apr 2004, John Coggeshall wrote: > { > +TSRMLS_FETCH(); > php_error_docref(NULL TSRMLS_CC, E_ERROR, "Could not allocate memory for tidy! > (Reason: %s)", (char *)msg); > } spaces => tabs ? and here too: > - TIDY_APPLY_CONFIG_ZVAL(obj->ptdoc->doc, options); > +if(