Re: [Cgi-session-user] proposed refinements to load() and find() interfaces
Brought to you by:
sherzodr
From: Matt L. <mle...@cp...> - 2006-06-10 21:15:00
|
On 6/10/06, *Mark Stosberg* <ma...@su...> wrote: > Here's another refactor: > > 1. Leave things generally alone with load(), but add extra param, > for internal use only, just be clearer than load/_load > > load($dsn,$query, \%dsn_args, $update_atime); > All right. _load's been ditched and load now accepts a fourth parameter. No documentation has been added about the fourth option and should be considered for internal use only. > 2. Leave find() mostly alone as well, and don't add update_atime flag. > Back out the coderef_args option patch in favor of documenting how > to use a higher order function for this. > > find( $dsn, > inspect( look => 'closely'), # will return a code ref > \%dsn_args ); Ok then. find is now changed back to three args (sorry Ron) but most of the documentation from Ron's patch remains. I've also updated the documentation to show how to create an anonymous sub to call a coderef with extra parameters. > Better? Yeah, if all tests pass for everyone else, let's release. -Matt |