off-topic check for division by 0 added

http://codereview.appspot.com/337560043


---

** [issues:#1493] Problem with horizontal beams**

**Status:** Started
**Created:** Wed Jan 26, 2011 12:21 PM UTC by Anonymous
**Last Updated:** Sat Feb 10, 2018 02:38 PM UTC
**Owner:** Torsten Hämmerle
**Attachments:**

- 
[beams.ly](https://sourceforge.net/p/testlilyissues/issues/1493/attachment/beams.ly)
 (807 Bytes; application/octet-stream)
- 
[screenshot.png](https://sourceforge.net/p/testlilyissues/issues/1493/attachment/screenshot.png)
 (18.9 kB; image/png)


*Originally created by:* *anonymous

*Originally created by:* 
[ralphbug...@gmail.com](http://code.google.com/u/106131861630194758622/)

James Lowe :
There seems to be an inconsistency with setting horizontal beams.

We have a snippet where we state that

\override Beam \#'damping = \#+inf.0

Should generate horizontal beams in all cases.

However the simple example attached shows some odd inconsistencies.

   \version "2.13.40"

\relative c''  \{
    \override Beam \#'damping = \#+inf.0
     f16 g a b    a c d g,    a b a c   d e f g, % all beams horizontal
     f16 g a b    a c d g,    a b a c   d e f g, % all beams horizontal
     f16 g a b    a c d g,    a b a c   d e f g, % 2nd and 4th group not 
Horizontal
     f16 g a b    a c d g,    a b a c   d e f g, % 2nd and 4th group not 
Horizontal
\}

\relative c''  \{
    \override Beam \#'damping = \#+inf.0
     f16 g a b    a c d g,    a b a c   d e f g, \break % 4th group not 
Horizontal
     f16 g a b    a c d g,    a b a c   d e f g, \break % 2nd and 4th group not 
Horizontal
     f16 g a b    a c d g,    a b a c   d e f g, \break % 2nd and 4th group not 
Horizontal
     f16 g a b    a c d g,    a b a c   d e f g, \break % 2nd and 4th group not 
Horizontal
\}

Phil Holmes :

I'd suggest the snippet is worth sorting out, too.  What do you reckon - adding 
the commands

   \override Beam \#'details \#'damping-direction-penalty = \#0
   \override Beam \#'details \#'round-to-zero-slope = \#0

to the existing snippet, or a new one pointed to by the old one, that says "if  
\override Beam \#'damping = \#+inf.0 doesn't do what you want, add the other 
commands too, as shown here"?


---

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
  • [Lilypond-... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto

Reply via email to