Code review comment for lp:~smspillaz/unity/unity.fix_881190_918360

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

On Fri, 20 Jan 2012, Daniel van Vugt wrote:

> Conflicts with: https://code.launchpad.net/~vanvugt/unity/fix-918329-trunk/+merge/89391
>
> Also, this proposal attempts to fix 2 separate bugs. I think you should remove the fix for 918360 which is redundant with my proposal and re-propose just a fix for bug 881190.

Ack, removing the cycle code and keeping the fix for the unminimize

> --
> https://code.launchpad.net/~smspillaz/unity/unity.fix_881190_918360/+merge/89181
> You are the owner of lp:~smspillaz/unity/unity.fix_881190_918360.
>

« Back to merge proposal