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

Issue 5909048: Reduce amount of consing (mostly replace reverse with reverse! where appropriate) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 3 months ago by dak
Modified:
7 years, 2 months ago
Reviewers:
CC:
lilypond-devel_gnu.org
Base URL:
http://git.savannah.gnu.org/gitweb/?p=lilypond.git/trunk/
Visibility:
Public.

Description

Reduce amount of consing (mostly replace reverse with reverse! where appropriate) There is one slightly non-trivial change in that it changes evaluation order in lily/page-breaking.cc @@ -552,7 +552,7 @@ Page_breaking::draw_page (SCM systems, SCM configuration, in I don't expect it to be problematic, but it might be worth checking more thoroughly than the rest.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -13 lines) Patch
M lily/articulations.cc View 1 chunk +1 line, -1 line 0 comments Download
M lily/key-engraver.cc View 1 chunk +1 line, -1 line 0 comments Download
M lily/line-spanner.cc View 1 chunk +1 line, -2 lines 0 comments Download
M lily/page-breaking.cc View 3 chunks +3 lines, -2 lines 0 comments Download
M lily/page-layout-problem.cc View 1 chunk +1 line, -1 line 0 comments Download
M lily/paper-book.cc View 2 chunks +5 lines, -5 lines 0 comments Download
M lily/system.cc View 1 chunk +1 line, -1 line 0 comments Download

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