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

Issue 559960060: Remove ly:lexer-keywords command

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 4 months ago by dak
Modified:
4 months, 3 weeks ago
Reviewers:
lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Remove ly:lexer-keywords command It has been unused since its inception. If it were to be retained, it would make more sense to remove its (sole) argument since it was effectively being ignored (other than that the presence of a Lexer guarantees that the keyword table has been initialised). Supporting member function Lily_lexer::keyword_list is removed alongside.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -42 lines) Patch
M lily/include/lily-lexer.hh View 1 chunk +0 lines, -1 line 0 comments Download
M lily/lily-lexer.cc View 1 chunk +0 lines, -9 lines 0 comments Download
D lily/lily-lexer-scheme.cc View 1 chunk +0 lines, -32 lines 0 comments Download

Messages

Total messages: 1
lemzwerg
1 year, 4 months ago (2020-05-03 05:49:06 UTC) #1
LGTM
Sign in to reply to this message.

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