Re: Re: cvs: php4 /ext/msession msession.c
| From: | Stig S. Bakken | Date: | Tue, 05 Feb 2002 13:04:09 +0000 |
| Subject: | Re: Re: cvs: php4 /ext/msession msession.c | ||
| References: | 1 2 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-79888@lists.php.net to get a copy of this message | ||
On Tue, 2002-02-05 at 10:36, Hartmut Holzgraefe wrote:
> mlwmohawk@mohawksoft.com wrote:
> > We have been down this road before. There is a reasonable expectation that
> > extensions may differ subtly from extension to extension. I see no need to
> > fork my source for 4.0.x and 4.1.x compatibility when it is simple enough to
> > do with macros and maintain one source file.
>
> ... and with #ifdefs all over the place it becomes a complete mess,
> even now with only two different api versions supported
>
> maintainance of older releases is what we have CVS branches for,
> we have more than enough #ifdef's in the code (due to configure detected
> stuff and threaded server support), i see no need to have even more
> of those in situations where we have other means to deal with
Well, shouldn't Mark, who maintains the code in question, be the one to
decide whether a couple of extra ifdefs makes his code unmaintainable?
- Stig