# ext/soap parse error in HEAD - Thread: https://externals.io/message/16213 - Original archive: https://news-web.php.net/php.internals/16213 - Started by Adam Maccabee Trachtenberg on 2005-05-11T17:56:46+00:00 - Messages: 8 - Reply by email to internals@lists.php.net - More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt --- ## #16213 — Adam Maccabee Trachtenberg — 2005-05-11T17:56:46+00:00 URL: https://externals.io/message/16213#16213 I'm getting a parse error with ext/soap in HEAD. This patch seems to fix my problem. Can someone review and apply? http://www.trachtenberg.com/patches/soap_error_handler.txt Thanks. -adam -- adam@trachtenberg.com | http://www.trachtenberg.com author of o'reilly's "upgrading to php 5" and "php cookbook" avoid the holiday rush, buy your copies today! --- ## #16215 — Antony Dovgal — 2005-05-11T19:18:17+00:00 URL: https://externals.io/message/16213#16215 · In reply to: #16213 On Wed, 11 May 2005 13:56:46 -0400 (EDT) Adam Maccabee Trachtenberg wrote: > I'm getting a parse error with ext/soap in HEAD. This patch seems to > fix my problem. Can someone review and apply? > > http://www.trachtenberg.com/patches/soap_error_handler.txt Are you sure that moving TSRMLS_FETCH() at the end of all declarations doesn't help? What compiler are you using? (gcc-3.3.4 works fine, with and without ZTS enabled). -- Wbr, Antony Dovgal aka tony2001 antony@zend.com --- ## #16216 — Adam Maccabee Trachtenberg — 2005-05-11T19:57:54+00:00 URL: https://externals.io/message/16213#16216 · In reply to: #16215 > On Wed, 11 May 2005 13:56:46 -0400 (EDT) > Adam Maccabee Trachtenberg wrote: > > > […] > > Are you sure that moving TSRMLS_FETCH() at the end of all declarations doesn't help? > What compiler are you using? (gcc-3.3.4 works fine, with and without ZTS enabled). That could be okay. I wasn't sure how TSRMLS_FETCH() effected the EG() and CG() macros. FWIW, I am using "gcc version 2.95.4 20011002 (Debian prerelease)". -adam -- adam@trachtenberg.com | http://www.trachtenberg.com author of o'reilly's "upgrading to php 5" and "php cookbook" avoid the holiday rush, buy your copies today! --- ## #16217 — Antony Dovgal — 2005-05-11T20:05:53+00:00 URL: https://externals.io/message/16213#16217 · In reply to: #16216 On Wed, 11 May 2005 15:57:54 -0400 (EDT) Adam Maccabee Trachtenberg wrote: > > > > […] > > That could be okay. I wasn't sure how TSRMLS_FETCH() effected the EG() > and CG() macros. > > FWIW, I am using "gcc version 2.95.4 20011002 (Debian prerelease)". Please try next snapshot (or update your local CVS repository), I committed the patch few seconds ago. -- Wbr, Antony Dovgal aka tony2001 antony@zend.com --- ## #16218 — l0t3k — 2005-05-11T20:31:07+00:00 URL: https://externals.io/message/16213#16218 · In reply to: #16215 Antony, i've had a similar issue with extensions i've written. i get "Undeclared identifer" problems under VS.NET if TSRMLS_FETCH() does not appear after all other declarations. l0t3k "Antony Dovgal" wrote in message news:20050511231817.08022b66.antony@zend.com... > On Wed, 11 May 2005 13:56:46 -0400 (EDT) > Adam Maccabee Trachtenberg wrote: > > > […] > > Are you sure that moving TSRMLS_FETCH() at the end of all declarations > doesn't help? > What compiler are you using? (gcc-3.3.4 works fine, with and without ZTS > enabled). > > -- > Wbr, > Antony Dovgal aka tony2001 > antony@zend.com --- ## #16222 — Rasmus Lerdorf — 2005-05-11T21:06:53+00:00 URL: https://externals.io/message/16213#16222 · In reply to: #16215 Antony Dovgal wrote: > On Wed, 11 May 2005 13:56:46 -0400 (EDT) > Adam Maccabee Trachtenberg wrote: > > > > […] > > > Are you sure that moving TSRMLS_FETCH() at the end of all declarations doesn't help? > What compiler are you using? (gcc-3.3.4 works fine, with and without ZTS enabled). Yes, just moving it to the end solves it on my old FreeBSD box. -Rasmus --- ## #16219 — Rob Richards — 2005-05-11T21:08:40+00:00 URL: https://externals.io/message/16213#16219 · In reply to: #16215 Ok, now I cant build under windows with that change. Rob Antony Dovgal wrote: >On Wed, 11 May 2005 13:56:46 -0400 (EDT) >Adam Maccabee Trachtenberg wrote: > > > > > […] > >Are you sure that moving TSRMLS_FETCH() at the end of all declarations doesn't help? >What compiler are you using? (gcc-3.3.4 works fine, with and without ZTS enabled). --- ## #16220 — Antony Dovgal — 2005-05-11T21:17:05+00:00 URL: https://externals.io/message/16213#16220 · In reply to: #16219 On Wed, 11 May 2005 17:08:40 -0400 Rob Richards wrote: > Ok, now I cant build under windows with that change. Ok, fixed. -- Wbr, Antony Dovgal aka tony2001 antony@zend.com