- Description has changed:
Diff:
~~~~
~~~~
- **Needs**: -->
- **Patch**: new --> review
- **Comment**:
Passes make, make check and a full make doc.
---
** [issues:#4961] Convert a bunch of C++ internals to degrees rather than
radians**
**Status:** Started
**Created:** Mon Aug 29, 2016 11:22 AM UTC by David Kastrup
**Last Updated:** Mon Aug 29, 2016 11:22 AM UTC
**Owner:** David Kastrup
Convert a bunch of C++ internals to degrees rather than radians
Degrees make for more "predictable" results in graphics applications
when care is taken to work as exactly as possible in the vicinity of
multiples of 45 degrees.
This provides also functions ly:length, ly:directed and ly:angle to
Scheme in order to facilitate work in degrees there. A number of
operations relevant only to complex analysis/arithmetic rather than
vector operations are removed from the Offset type in order not to
encourage their use.
Contains commits:
Remove complex{conjugate,divide,exp}, Offset::arg
They are promoting aspects of Offset not well-suited to graphics.
Let make_partial_ellipse_boxes use degrees
Change rotations to degrees rather than radians
Or replace them with something not requiring angles altogether.
Simplify polar->rectangular
Add ly:length, ly:directed, ly:angle
Those are functions working on and/or producing coordinate pairs, usually
based on angles expressed in degrees.
Add offset_directed (Real)
This converts an angle in degrees into a unit vector.
Make Offset::angle_degrees () deliver nicer angles
Multiples of 45 degrees should be returned exactly.
http://codereview.appspot.com/305380043
---
Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.------------------------------------------------------------------------------
_______________________________________________
Testlilyissues-auto mailing list
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
