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

Issue 553980044: Further cleanup/regularisation of Callback_wrapper classes and trampolines (Closed)

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

Description

Further cleanup/regularisation of Callback_wrapper classes and trampolines Contains two commits: Callback_wrapper:: move trampolines and make_smob to Callbacks class Make Callback_wrapper a class template Templating Callback_wrapper on the wrapped argument list avoids a lot of code repetition.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+92 lines, -109 lines) Patch
M lily/context.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M lily/include/callback.hh View 5 chunks +43 lines, -65 lines 0 comments Download
M lily/include/engraver.hh View 2 chunks +13 lines, -12 lines 0 comments Download
M lily/include/listener.hh View 1 chunk +1 line, -1 line 0 comments Download
M lily/include/smobs.hh View 1 chunk +5 lines, -4 lines 0 comments Download
M lily/include/translator.hh View 2 chunks +15 lines, -12 lines 0 comments Download
M lily/include/translator.icc View 4 chunks +7 lines, -7 lines 0 comments Download
M lily/score-engraver.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M lily/score-performer.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M lily/translator-group.cc View 1 chunk +2 lines, -2 lines 0 comments Download

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