On Fri, Jun 01, 2001 at 10:25:51AM +0100, Graham Barr wrote:
> On Fri, Jun 01, 2001 at 10:23:39AM +0100, Graham Barr wrote:
> > On Fri, Jun 01, 2001 at 10:19:51AM +0100, Tim Bunce wrote:
> > > Thanks. I'll fix it for the next release. Meanwhile try
> > > 
> > >   ... $attr ? sort %$attr : ();
> > 
> > Um, that will mix the kesy and values. You would need something like
> > 
> >   ... $attr ? map { ($_,$attr->{$_}) } sort keys %$attr : ()
> 
> Ignore me, it does not really matter in this case. I'm just having a bad day
> and its nove even 10:30 am :)

Actually, I think it does matter, and I was going to point it out myself.

( RaiseError => 1, AutoCommit => 0 )
( RaiseError => 0, AutoCommit => 1 )

Those sort to the same list:

( 0, 1, AutoCommit, RaiseError )

So connect_cached() could return a handle with the wrong attributes.


Am I missing something?


Ronald

Reply via email to