Review: Approve

282     +                    throw err;
289     +                    throw err;
293     +                    throw err;

Seems like it could be moved out of the ifs.

Other than that it's fine.
-- 
https://code.launchpad.net/~zeitgeist/zeitgeist/dbfails/+merge/87195
Your team Zeitgeist Framework Team is subscribed to branch lp:zeitgeist.

_______________________________________________
Mailing list: https://launchpad.net/~zeitgeist
Post to     : zeitgeist@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zeitgeist
More help   : https://help.launchpad.net/ListHelp

Reply via email to