Code review comment for lp:~verterok/ubuntuone-client/fix-598189

Revision history for this message
Facundo Batista (facundo) wrote :

I would use local vars for self.fsm.changed and self.fsm.CHANGED_SERVER instead of accessing self and fsm (twice) inside the double for.

Otherwise, it's ok.

review: Approve

« Back to merge proposal