[PATCH] add basegfx::fround to unittest (was: basegfx::fround and friends seems needed in windows build)

Korrawit Pruegsanusak <detective.conan.1412 -AT- gmail.com>
Tue, 21 Feb 2012 21:24:20 +0700

Hello Michael, Stephan, all,

First, sorry for my late reply. I've been busy last week. :(

On Mon, Feb 13, 2012 at 18:24, Michael Meeks <michael.meeks@suse.com> wrote:

I suggest we add a method to test/basegfx2d.cxx that has an equivalent:


Thanks for your suggestion :)
Now I attach a patch for reviewing.

OK, I have to confess that I've shamelessly copy-n'-pasted the whole
things from the adjacent ones, and made some modifications on top of
it.

From reading the b2drange.hxx, I think this method is to round the
values of the range from double to integer. So I just check a simple
value rounding.

The `cd basegfx && make check` (as Stephan pointed) works fine. And if
I change the integer value from (1,2,4,5) to (1,2,4,4), the test
failed.

If it's ok, I could push it to master.
But if I've done something wrong, please feel free to correct me. :)

Best Regards,
-- 
Korrawit Pruegsanusak

Attachment: add-fround-to-unit-test.patch
Description: Binary data

Context

[PATCH] add basegfx::fround to unittest (was: basegfx::fround and friends seems needed in windows build) · Korrawit Pruegsanusak
- [PUSHED] add basegfx::fround to unittest (was: basegfx::fround and friends seems needed in windows build) · Michael Meeks
  - Re: [PUSHED] add basegfx::fround to unittest (was: basegfx::fround and friends seems needed in windows build) · Korrawit Pruegsanusak

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.