> I don't need to fully understand this code to say: > > 1. The change to src/window.cpp looks like a mistake so should be removed.
Ack
> > 2. Are you sure this proposal fixes bug 891591? Or is that a typo and its > meant to fix a different bug?
Right, it should be bug 896591 . That was a typo
« Back to merge proposal
> I don't need to fully understand this code to say:
>
> 1. The change to src/window.cpp looks like a mistake so should be removed.
Ack
>
> 2. Are you sure this proposal fixes bug 891591? Or is that a typo and its
> meant to fix a different bug?
Right, it should be bug 896591 . That was a typo