Perhaps I don't have right to review patch on codereview site, however patch
looks good to me.
---
** [issues:#5320] beam.cc: avoid calculating an invalid reference**
**Status:** Started
**Created:** Wed May 09, 2018 11:49 AM UTC by David Kastrup
**Last Updated:** Wed May 09, 2018 01:01 PM UTC
**Owner:** David Kastrup
beam.cc: avoid calculating an invalid reference
In calc_beam_segments, an invalid reference to a neighboring segment
was calculated (and not used) at the extreme ends of the beam. This
caused segfaults on Fedora. Since the reference is used at most twice
anyway and in the same expression, it seems easier to just use the
expression directly (the invalid cases are then skipped due to
short-circuit evaluation).
http://codereview.appspot.com/343160043
---
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.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Testlilyissues-auto mailing list
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto