Re: [PHPLIB] phplib - changes proposal From: Teodor Cimpoesu (teo <email protected>)
Date: 10/27/99

Kristian Köhntopp wrote:
>
> Teodor Cimpoesu wrote:
> > Also the encoding schema (CT_Sql::ac_get_value,ac_store) uses heavily
> > ereg()s which I didn't like so change it to a simply enc_meth:string
> > schema.
>
> I have had a look at your changes to ct_sql.inc and they look sensible.
> I have included them and will test them.
k,
 
> I have rejected your changes to session.inc on the grounds that they use
> up additional global variables. This is already a problem with current
> PHPLIB, and I will not accept code that uses additional globals. I
you mean $_in and $_pt ? okay , no big problem with that

> understand why you have to use that variables when you do things
> the way you propose, but I feel that this simply isn't right with
> current PHPLIB.
Okay, but I would apreciate to say what's wrong. You mean using other
globals variables or just the way I wrote the serialize() ?
 
> Perhaps with a (up to now hyptotheitcal) session4.inc and using the
> PHP4 builting session functions...
:) looking fws
 
> Also, please subscribe to phplib-dev <email protected>, our developer
> mailing list, and take this stuff there.
I did
so here is a diff to my session.inc version to strip out the globals
variables

email protected>?subject=Re:%20[PHPLIB]%20phplib%20-%20changes%20proposal&replyto=3816F191.3D489FFF <email protected>">guest <email protected>/devel/php/phplib-7/out/session.inc.diff2">http://guest:guest <email protected>/devel/php/phplib-7/out/session.inc.diff2
[ just dropping those globals ] 

-- 
CIMPOESU Teodor, Web Programmer

@ DIGICOM S.A. Bucharest, Romania
@ Internet, site development
@ teo <email protected>,+(401)-330.47.28

official home page ~ http://www.digiro.net/
Internet  web page ~ http://internet.digiro.net/
-
PHP3 Base Library Mailing List. Send messages to <phplib <email protected>>.
To unsubscribe, send "unsubscribe" to <phplib-request <email protected>> in
the body, not the subject, of your message.