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

Issue 314440043: Fix a number of SCM/int confusions. (Closed)

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

Description

Fix a number of SCM/int confusions. Contains commits: Stop SCM/int confusion in stencil head registration Stop SCM/int confusion in program-option code Stop SCM/int confusion in Drum_notes_engraver Stop SCM/int confusion for grob interface table Stop SCM/int confusion in function documentation tables Stop SCM/int confusion for property profiling Use Protected_scm.is_bound () with Scheme engravers Add Protected::is_bound () function Since SCM_UNBNDP has problems with arguments of type Protected_scm, an alternative member function seems like a viable alternative.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+57 lines, -49 lines) Patch
M lily/drum-note-engraver.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M lily/function-documentation.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M lily/grob-interface-scheme.cc View 1 chunk +4 lines, -7 lines 0 comments Download
M lily/include/profile.hh View 1 chunk +6 lines, -4 lines 0 comments Download
M lily/include/protected-scm.hh View 1 chunk +1 line, -0 lines 0 comments Download
M lily/profile.cc View 2 chunks +16 lines, -17 lines 0 comments Download
M lily/program-option-scheme.cc View 3 chunks +4 lines, -3 lines 0 comments Download
M lily/protected-scm.cc View 1 chunk +8 lines, -0 lines 0 comments Download
M lily/stencil-expression.cc View 2 chunks +4 lines, -7 lines 0 comments Download
M lily/translator-ctors.cc View 2 chunks +6 lines, -4 lines 0 comments Download

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