Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1664)

Issue 7346046: Replace several misuses of eq? on numerical values (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 2 months ago by dak
Modified:
11 years, 2 months ago
Reviewers:
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Replace several misuses of eq? on numerical values Numerical values don't carry identity, so (eq? 0 0) may or may not evaluate to #t. Those need to be replaced by eqv? or, where both items are guaranteed to be a number, = .

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -19 lines) Patch
M Documentation/snippets/transposing-pitches-with-minimum-accidentals-smart-transpose.ly View 1 chunk +2 lines, -2 lines 0 comments Download
M input/regression/glissando-index.ly View 1 chunk +1 line, -1 line 0 comments Download
M ly/event-listener.ly View 1 chunk +1 line, -1 line 0 comments Download
M scm/define-event-classes.scm View 1 chunk +1 line, -1 line 0 comments Download
M scm/define-markup-commands.scm View 3 chunks +4 lines, -4 lines 0 comments Download
M scm/stencil.scm View 2 chunks +2 lines, -2 lines 0 comments Download
M scm/translation-functions.scm View 6 chunks +8 lines, -8 lines 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b