Review: Approve

Nice change, introducing preloadPOFilesAndSequence really solves the problem 
for loading suggestions for a given message.

We can land it ASAP.

(thinking out loud here, feel free to ignore me)

Although, I am not seeing a clear path for going further and pre-loading this 
for multiple messages. Maybe we won't need it (this step will bring enough 
benefits) or we could try to investigate something at the model level for 
making it possible.

-- 
https://code.launchpad.net/~wgrant/launchpad/tm-suggest-constant/+merge/225260
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

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

Reply via email to