[Dbi-interbase-devel] bug in set_txn_param

[Mark D. A. <md...@di...>]

first off i'm so glad there is now the ability to set transaction parameters (as an archive
search will show; i've been campaigning for a while :)).

there is a missing comma in this documentation in InterBase.pm:
 $dbh->func( 
    -isolation_level => ['read_committed', 'record_version']
    'set_tx_param'
 );

also, this works for me:
 $dbh->func(
     -access_mode => 'read_write',
     -isolation_level => 'read_committed', 
     -lock_resolution => 'wait', 
     'set_tx_param');

but this change gets me the spurious error "Invalid -isolation_level value":
   -isolation_level => ['read_committed'],

and this change gets me a perl crash:
  -isolation_level => ['read_committed', 'record_version'],

looking at the code in InterBase.xs, it is not obvious to me what is wrong.
rc is not reset to zero in the outer loop, but that should not affect this case.
there also seems to be some unsafe assumptions about sv being a valid string,
but i'm not enough experienced with xs to say.

this is with 28.2 on win32

-mda