improving empty handling in tools::Rectangle

[Chris Sherlock]

> On 9 May 2019, at 6:23 am, Thorsten Behrens <thb at libreoffice.org> wrote:
> 
> Hi Noel, *,
> 
> Noel Grandin wrote:
>> What I'm doing here is making its behaviour in the empty case more reasonable,
>> and adding asserts that will flush out some of the existing dodgy code.
>> 
> Yep, on balance I think that's beneficial.
> 
>> I'd be happy to be proven wrong, but I'm not aware of any mechanical
>> fixes.
>> 
> Right, and I'm afraid those would obfuscate things even more.
> 
> The plan back in the day was to either use float ranges (where - for
> graphics - this half vs. closed interval question usually becomes
> moot), or B2IBox and B2IRange respectively. And then gradually move
> code over to use basegfx & drawinglayer.
> 
> From how I see this, that change is best done manual.

Can we file an easy hack to manually start changing to either half or closed intervals (whatever basegfx does)?

Chris 

> 
> Cheers,
> 
> -- Thorsten