[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5068 SCM/bool confusion in One_page_breaking::solve

[Auto mailings of changes to Lily Issues]

• 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 address@hidden
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 address@hidden 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
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto