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

Issue 238670043: Issue 4427: clean up finding & changing of contexts in C++ (Closed)

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

Description

clean up finding & changing of contexts in C++ Combine the key feature of Change_iterator and Auto_change_iterator into a common function (favoring the logic of Change_iterator). Define functions for various context-search loops.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+128 lines, -146 lines) Patch
M lily/accidental-engraver.cc View 1 chunk +1 line, -4 lines 0 comments Download
M lily/auto-change-iterator.cc View 4 chunks +6 lines, -55 lines 0 comments Download
M lily/change-iterator.cc View 1 chunk +43 lines, -51 lines 0 comments Download
M lily/context.cc View 2 chunks +43 lines, -3 lines 0 comments Download
M lily/grace-engraver.cc View 1 chunk +1 line, -4 lines 0 comments Download
M lily/include/change-iterator.hh View 1 chunk +4 lines, -0 lines 0 comments Download
M lily/include/context.hh View 1 chunk +24 lines, -0 lines 0 comments Download
M lily/lyric-combine-music-iterator.cc View 1 chunk +2 lines, -6 lines 0 comments Download
M lily/lyric-engraver.cc View 1 chunk +2 lines, -17 lines 0 comments Download
M lily/note-head-line-engraver.cc View 1 chunk +2 lines, -6 lines 0 comments Download

Messages

Total messages: 1
Dan Eble
8 years, 11 months ago (2015-05-31 20:48:20 UTC) #1

          
Sign in to reply to this message.

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