Menu

#3 Replace hard-coded math constants with cmath constants

Unstable (example)
closed-accepted
nobody
None
5
2023-08-21
2019-01-28
Rob Gill
No

agg_trans_affine.h uses hard-coded math constants for 1/sqrt(2), a more precise value for this constant if provided by cmath's M_SQRT1_2.

Proposed patch included.

1 Attachments

Discussion

  • John Horigan

    John Horigan - 2021-02-09

    M_SQRT1_2 is from the POSIX standard, not any C or C++ standard. I can update the file with a more precise version of the constant.

     
  • Jim Barry

    Jim Barry - 2023-08-21
    • status: open --> closed-accepted
     
  • Jim Barry

    Jim Barry - 2023-08-21

    Addressed in r136.

     

Log in to post a comment.