diff options
author | Thomas White <taw@physics.org> | 2010-03-03 15:03:07 +0100 |
---|---|---|
committer | Thomas White <taw@physics.org> | 2010-03-03 15:03:07 +0100 |
commit | 851c205e1a3d2bb059b3e045a73682eed0f605a3 (patch) | |
tree | f570c4961f290a7c4221afc5123821db25ccd2f9 /src/utils.h | |
parent | bb9c12bb456f750ae465b9c5a6c1ec47f3f8fb08 (diff) |
Fix domain of acos in angle_between()
Diffstat (limited to 'src/utils.h')
-rw-r--r-- | src/utils.h | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/utils.h b/src/utils.h index d9025fa0..7d8274bd 100644 --- a/src/utils.h +++ b/src/utils.h @@ -107,7 +107,13 @@ static inline double angle_between(double x1, double y1, double z1, { double mod1 = modulus(x1, y1, z1); double mod2 = modulus(x2, y2, z2); - return acos( (x1*x2 + y1*y2 + z1*z2) / (mod1*mod2) ); + double cosine = (x1*x2 + y1*y2 + z1*z2) / (mod1*mod2); + + /* Fix domain if outside due to rounding */ + if ( cosine > 1.0 ) cosine = 1.0; + if ( cosine < -1.0 ) cosine = -1.0; + + return acos(cosine); } |