Review: Needs Fixing

261-274 This double loop seems inefficient, with thousands of labels, it will 
loop over each of them and do an additional query even if the label has nothing 
to do with the statement. You should rather loop on the statement lines to 
complete and query the labels in the inner loop.

Thanks for your proposal
-- 
https://code.launchpad.net/~akretion-team/banking-addons/bank-statement-reconcile-7.0-add-completion-label/+merge/197758
Your team Banking Addons Core Editors is subscribed to branch 
lp:banking-addons/bank-statement-reconcile-7.0.

-- 
Mailing list: https://launchpad.net/~banking-addons-team
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~banking-addons-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to