Re: DB/ibase.php changes

thanks Francisco, I committed the freeResult() patch (the other one as you said was already in CVS) Ludo ----- Original Message ----- From: "Francisco Benza" <franbenz@roshka.com.py> To: <pear-dev@lists.php.net> Sent: Tuesday, September 24, 2002 10:50 PM Subject: [PEAR-DEV] DB/ibase.php changes > Hi > > I've changed two parts of the ibase.php file(v 1.3 2002/05/09 12:29:53 ): > 1) to allow windows database path(already addressed in v1.4); > 2) to free results resources(the freeResult function wasn't receiving > the $result parameter because it wasn't in it's parameter list)(this > caused me a lot of stress). > > Francisco Benza. > ---------------------------------------------------------------------------- ---- > --- DB/ibase.php Tue Sep 24 15:39:06 2002 > +++ DB/ibase.php.new Tue Sep 24 17:32:08 2002 > @@ -79,7 +79,7 @@ > $user = $dsninfo['username']; > $pw = $dsninfo['password']; > $dbhost = $dsninfo['hostspec'] ? > - ($dsninfo['hostspec'] . ':/' . > $dsninfo['database']) : > + ($dsninfo['hostspec'] . ':' . > $dsninfo['database']) : > $dsninfo['database']; > > $connect_function = $persistent ? 'ibase_pconnect' : 'ibase_connect'; > @@ -206,7 +206,7 @@ > return DB_OK; > } > > - function freeResult() > + function freeResult($result) > { > if (is_resource($result)) { > return ibase_free_result($result); > > ---------------------------------------------------------------------------- ---- > -- > PEAR Development Mailing List (http://pear.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php