Code review comment for lp:~dandrader/frame/smarter_backend

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

> * Any functions that may fail need to return a UFStatus and provide the return
> result through a referenced parameter. All of the new functions here should do
> this. I forgot about this when I approved the previous merge proposal as well.
> We should go back and fix these up.

Isn't that extra status variable redundant in those cases? What's the benefit?

« Back to merge proposal