Code review comment for lp:~ted/ubuntu-app-launch/second-exec-test

Revision history for this message
Charles Kerr (charlesk) wrote :

Looks very good! I only have a couple of trivial issues.

- if you're going to init resume_timeout to 0 in its declaration, you need to pass -std=c++11 to gcc

- last_resume_appid and last_focus_appid are leaked when the test ends -- a std::string might be better here

« Back to merge proposal