Review Request 129358: Return SurfaceInterface::size in surface normal co-ordinates

Martin Gräßlin mgraesslin at kde.org
Fri Nov 11 06:30:21 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129358/#review100769
-----------------------------------------------------------



unrelated: you could have used phab for the request :-)

I'm not completely sure whether it's a good idea to change the semantics of the method, on the other hand I seem to have added a todo for that in the past.


src/server/surface_interface.h (line 117)
<https://git.reviewboard.kde.org/r/129358/#comment67643>

    "in surface normal"? What do you mean by that?



src/server/surface_interface.cpp (lines 338 - 340)
<https://git.reviewboard.kde.org/r/129358/#comment67644>

    please add a test case to test this condition.



src/server/surface_interface.cpp (line 642)
<https://git.reviewboard.kde.org/r/129358/#comment67645>

    this should also be covered in the autotests


- Martin Gräßlin


On Nov. 10, 2016, 4:56 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129358/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2016, 4:56 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kwayland
> 
> 
> Description
> -------
> 
> Return SurfaceInterface::size in surface normal co-ordinates
> 
> 
> Diffs
> -----
> 
>   src/server/surface_interface.h de74ef336aa7d17c1bd95ba453b8d4f620e416ec 
>   src/server/surface_interface.cpp 157907210254c550e85d24eaa19637f933872ea7 
> 
> Diff: https://git.reviewboard.kde.org/r/129358/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161111/36451745/attachment.html>


More information about the Plasma-devel mailing list