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

Issue 7020044: Simplify several library functions. (Closed)

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

Description

Simplify several library functions. Most of those do not even seem to be used. Maybe one should instead remove them instead of having to maintain bit-rot prone stuff. Simplify hash-table->alist, alist->hash-table, functional-or, functional-and, list-element-index Simplify first-member and first-assoc

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -27 lines) Patch
M scm/lily-library.scm View 3 chunks +7 lines, -27 lines 0 comments Download

Messages

Total messages: 3
lemzwerg
Everything looks very nice! I just wonder how it comes that those simplifications haven't been ...
11 years, 4 months ago (2012-12-27 19:07:15 UTC) #1
thomasmorley65
LGTM Two questions: a) Where is `identity´ defined? I can't find it, neither in LilyPond ...
11 years, 4 months ago (2012-12-28 15:09:24 UTC) #2
dak
11 years, 4 months ago (2012-12-28 15:48:56 UTC) #3
On 2012/12/28 15:09:24, thomasmorley65 wrote:
> LGTM
> 
> Two questions:
> a) Where is `identity´ defined? I can't find it, neither in LilyPond nor in
the
> guile-manual.

Huh.  The Guile2 manual shows its definition.  It's a common function and is
equivalent to (lambda (x) x).
I had not been aware that its documentation is deficient in Guile1.

> b) Why are the definitions in lily-library.scm not all (or nearly all) defined
> public?
> Some examples:
>   split-list
>   other-axis
>   index-cell
>   sign
> 
> I see no reason _not_ to make them public.

Actually, that would be the perfect reason to throw them out when they are not
being used in our current code base.  Apart from "other-axis", they are not
really specific to LilyPond and don't offer significant advantages over the
standard Guile libraries.
Sign in to reply to this message.

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