Mir

Code review comment for lp:~cemil-azizoglu/mir/fix-1590959

Revision history for this message
Kevin DuBois (kdub) wrote :

+ auto const border = 150; //must match the border value in clip_to_display()
Redundant comment? If the production code changes, the test will tell us when run.

nit:
+ if ((screen_width < requested_size.width.as_uint32_t()+border) ||
+ (screen_height < requested_size.height.as_uint32_t()+border))
spaces between +

review: Approve

« Back to merge proposal