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

Issue 236830043: Reorganize smob initialization to make it more reliable (Closed)

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

Description

Reorganize smob initialization to make it more reliable This change is a nuisance in that it requires an explicit call to ADD_SMOB_INIT for each Scheme-related class. However, this setup guarantees that at the point Scheme execution commences, all predicates of the likes of "ly:grob?" will be present. It also guarantees that if a particular type should seem required during _other_ initializations that happen to depend on the type and get executed earlier, it will be provided early.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+96 lines, -35 lines) Patch
M lily/book.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/box.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/context.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/context-def.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/context-mod.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/context-property.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/dispatcher.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/duration.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/font-metric.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/grob.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/grob-array.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/include/smobs.hh View 2 chunks +21 lines, -21 lines 0 comments Download
M lily/include/smobs.tcc View 5 chunks +8 lines, -14 lines 0 comments Download
M lily/input-smob.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/lily-lexer.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/lily-parser.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/listener.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/moment.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/music-function.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/music-iterator.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/music-output.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/output-def.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/page-marker.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/paper-book.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/paper-outputter.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/pitch.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/prob.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/scale.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/scheme-listener.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/scm-hash.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/score.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/simple-closure.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/simple-spacer.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/skyline.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/skyline-pair.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/source-file.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/spring.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/stencil.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/translator.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/translator-dispatch-list.cc View 1 chunk +1 line, -0 lines 0 comments Download
M lily/translator-group.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M lily/undead.cc View 1 chunk +2 lines, -0 lines 0 comments Download

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