0002-Change-_cairo_fixed_from_double-to-use-the-magic-number-technique.patch 2.7 KB

12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879
  1. From nobody Mon Sep 17 00:00:00 2001
  2. From: Dan Amelang <dan@amelang.net>
  3. Date: Sun Oct 29 21:31:23 2006 -0800
  4. Subject: [PATCH] Change _cairo_fixed_from_double to use the "magic number" technique
  5. See long thread here:
  6. http://lists.freedesktop.org/archives/cairo/2006-October/008285.html
  7. ---
  8. src/cairo-fixed.c | 48 +++++++++++++++++++++++++++++++++++++++++++++++-
  9. 1 files changed, 47 insertions(+), 1 deletions(-)
  10. d88acddcabe770e17664b34a2d5f74d3926e1642
  11. diff --git a/src/cairo-fixed.c b/src/cairo-fixed.c
  12. index 604c9e7..fe6c2dc 100644
  13. --- a/src/cairo-fixed.c
  14. +++ b/src/cairo-fixed.c
  15. @@ -42,10 +42,56 @@ _cairo_fixed_from_int (int i)
  16. return i << 16;
  17. }
  18. +/* This is the "magic number" approach to converting a double into fixed
  19. + * point as described here:
  20. + *
  21. + * http://www.stereopsis.com/sree/fpu2006.html (an overview)
  22. + * http://www.d6.com/users/checker/pdfs/gdmfp.pdf (in detail)
  23. + *
  24. + * The basic idea is to add a large enough number to the double that the
  25. + * literal floating point is moved up to the extent that it forces the
  26. + * double's value to be shifted down to the bottom of the mantissa (to make
  27. + * room for the large number being added in). Since the mantissa is, at a
  28. + * given moment in time, a fixed point integer itself, one can convert a
  29. + * float to various fixed point representations by moving around the point
  30. + * of a floating point number through arithmetic operations. This behavior
  31. + * is reliable on most modern platforms as it is mandated by the IEEE-754
  32. + * standard for floating point arithmetic.
  33. + *
  34. + * For our purposes, a "magic number" must be carefully selected that is
  35. + * both large enough to produce the desired point-shifting effect, and also
  36. + * has no lower bits in its representation that would interfere with our
  37. + * value at the bottom of the mantissa. The magic number is calculated as
  38. + * follows:
  39. + *
  40. + * (2 ^ (MANTISSA_SIZE - FRACTIONAL_SIZE)) * 1.5
  41. + *
  42. + * where in our case:
  43. + * - MANTISSA_SIZE for 64-bit doubles is 52
  44. + * - FRACTIONAL_SIZE for 16.16 fixed point is 16
  45. + *
  46. + * Although this approach provides a very large speedup of this function
  47. + * on a wide-array of systems, it does come with two caveats:
  48. + *
  49. + * 1) It uses banker's rounding as opposed to arithmetic rounding.
  50. + * 2) It doesn't function properly if the FPU is in single-precision
  51. + * mode.
  52. + */
  53. +#define CAIRO_MAGIC_NUMBER_FIXED_16_16 (103079215104.0)
  54. cairo_fixed_t
  55. _cairo_fixed_from_double (double d)
  56. {
  57. - return (cairo_fixed_t) floor (d * 65536 + 0.5);
  58. + union {
  59. + double d;
  60. + int32_t i[2];
  61. + } u;
  62. +
  63. + u.d = d + CAIRO_MAGIC_NUMBER_FIXED_16_16;
  64. +#ifdef FLOAT_WORDS_BIGENDIAN
  65. + return u.i[1];
  66. +#else
  67. + return u.i[0];
  68. +#endif
  69. }
  70. cairo_fixed_t
  71. --
  72. 1.2.6