Code review comment for lp:~mterry/qtmir/fix-wakelocks

Revision history for this message
Michael Terry (mterry) wrote :

> In Application::setInternalState(Application::InternalState state):
> [snip]

True. That whole switch statement could use a lot of cleanup as there are several duplicate entries. But it looks like an intentional choice on Gerry's part, and this change in this branch is really just a revert to bring this code back to what it looked like.

I can clean up if you really want it more slim. But I'd prefer to either (a) clean up the whole switch or (b) leave this code as it was. With a preference for (b).

« Back to merge proposal