diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-07-12 12:06:41 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-07-12 17:04:02 +0200 |
| commit | 1ce1c26dd98e6477139e08d1ebe89fa950ff5fb0 (patch) | |
| tree | 462fdc3beda919e1b399d8bb168f472cf175a48f /include/tools | |
| parent | b11ea5e9c37b19f0d60a4075146668954a7bf728 (diff) | |
make tools::Rectangle::Right return Left when empty
I tried making this assert, but there are just too many places where we
pass around empty rectangles, so rather just return the value of nLeft,
in a similar fashion to methods like Rectangle::TopLeft
Change-Id: I3377071ecae26f13e895ae411cd269f0bdbe0ef6
Reviewed-on: https://gerrit.libreoffice.org/75486
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'include/tools')
| -rw-r--r-- | include/tools/gen.hxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/tools/gen.hxx b/include/tools/gen.hxx index 54a438bfa161..dd0514396567 100644 --- a/include/tools/gen.hxx +++ b/include/tools/gen.hxx @@ -386,7 +386,7 @@ public: Rectangle( const Point& rLT, const Size& rSize ); long Left() const { return nLeft; } - long Right() const { return nRight; } + long Right() const; long Top() const { return nTop; } long Bottom() const { return nBottom; } |