ap / DBIx-Connector

Fast, safe DBI connection and transaction management
https://metacpan.org/release/DBIx-Connector
40 stars 14 forks source link

Use Try::Tiny in docs that reference removed catch sub #18

Closed rwstauner closed 12 years ago

rwstauner commented 12 years ago

Reading the docs (the Rollback Exceptions section) I saw a section that passed two subs, the second appearing to be an error handler.

I was confused until I read in the change log that there used to be a 'catch' block, so I'm assuming this pod was leftover from that change.

I wrapped the subs in a try/catch (using Try::Tiny).

If this is not correct, please review the docs to clarify. Thanks!

theory commented 12 years ago

Merged, thanks for the catch!