- **labels**: --> Fixed_2_19_56
- **status**: Started --> Fixed
- **Patch**: push -->
- **Type**: Enhancement --> Defect
- **Comment**:
Discussion with Paul seemed to indicate that pushing this seems like a better
option than not pushing it, so pushed to staging as
commit b27fe425c189263be20830b09c859d27245bb550
Author: David Kastrup <[email protected]>
Date: Tue Feb 14 13:33:13 2017 +0100
Issue 5068: SCM/bool confusion in One_page_breaking::solve
This replaces an accidentally always-true condition with the
originally intended one.
---
** [issues:#5068] SCM/bool confusion in One_page_breaking::solve**
**Status:** Fixed
**Labels:** Fixed_2_19_56
**Created:** Tue Feb 14, 2017 12:35 PM UTC by David Kastrup
**Last Updated:** Mon Feb 20, 2017 01:33 PM UTC
**Owner:** David Kastrup
SCM/bool confusion in One_page_breaking::solve
This replaces an accidentally always-true condition with the
originally intended one.
I have no idea about the consequences: it would be best if the
original author Paul Morris (in issue 4752) checked that this now
works as intended and also tried figuring out why we failed to notice
this problem.
http://codereview.appspot.com/314490043
---
Sent from sourceforge.net because [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
