Re: [PHP-CVS] cvs: php4 /ext/swf swf.c

2003-01-18 Thread Ilia A.
On January 18, 2003 03:20 pm, Marcus Börger wrote:
 At 20:57 18.01.2003, you wrote:
 iliaa   Sat Jan 18 14:57:25 2003 EDT
 
Modified files:
  /php4/ext/swf   swf.c
Log:
Changed php_error to php_error_docref.

 Thanks for all this work and one question: did you use my script to do it?

Nope, I've done it manually, I was mostly going after memory allocation thing 
and in some cases did php_error - php_error_docref along the way. My next 
goal is to remove some 690 php_error() form ext/ and about 100 or so 
zend_error(). So, if you have any automated tools, they'd be of great help.

Ilia

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php




Re: [PHP-CVS] cvs: php4 /ext/swf swf.c

2003-01-18 Thread Marcus Börger
At 21:35 18.01.2003, Ilia A. wrote:

My next
goal is to remove some 690 php_error() form ext/ and about 100 or so
zend_error(). So, if you have any automated tools, they'd be of great help.

Ilia


I meant the php_error thing and for that i have a script:
http://marcus-boerger.de/php/ext/docref.txt

I hope that helps :-)

marcus

p.s: Wow excluding google bot it was downloaded 300 times...121 times
referred from http://www.zend.com/zend/week/week112.php - iguess avin
was a bit missleading in his summary.



--
--
Marcus Börger - Looking for all sorts of freelance work - just ask...

Did i help you? Consider a gift:
http://www.amazon.de/exec/obidos/wishlist/ho722v0rg1u0
--



Re: [PHP-CVS] cvs: php4 /ext/swf swf.c

2003-01-18 Thread Marcus Börger
At 21:35 18.01.2003, Marcus Börger wrote:

At 21:35 18.01.2003, Ilia A. wrote:

My next
goal is to remove some 690 php_error() form ext/ and about 100 or so
zend_error(). So, if you have any automated tools, they'd be of great help.

Ilia


I meant the php_error thing and for that i have a script:
http://marcus-boerger.de/php/ext/docref.txt

I hope that helps :-)

marcus

p.s: Wow excluding google bot it was downloaded 300 times...121 times
referred from http://www.zend.com/zend/week/week112.php - iguess avin
was a bit missleading in his summary.


You can change all occurences when
a) you skip main/main.c
b) you use two sed commands in one go:
cat ${i} |
sed 
s/\(php_error\|zend_error\)\([[:space:]]*(\)\([^\]*\\)\(%s()\:\?[[:space:]]*\)\(.*\)\(,[[:space:]]*get_active_function_name[[:space:]]*([[:space:]]*TSRMLS_C[[:space:]]*)\)/php_error_docref(NULL 
TSRMLS_CC, \3\5/g | \
sed 
s/\(php_error\|zend_error\)\([[:space:]]*([[:space:]]*\)/php_error_docref(NULL 
TSRMLS_CC, /g \
 $i.tmp

sorry i do not have the time to make a new skrip. maybe tomorrow or on 
monday if you cna wait...

marcus



--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php