Code review comment for lp:~alan-griffiths/miral/fix-FTBFS-on-Xenial-with-Mir-0.27

Revision history for this message
Gerry Boland (gerboland) wrote :

Compiles ok!

Nitpicking: EXPECT_THAT(cookie, Ne(nullptr)) could be EXPECT_NE(nullptr, cookie);
(Google also seems to want the expected value to be first)

review: Approve

« Back to merge proposal