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

Issue 569340043: Remove unused ly_hash_scm (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 months, 2 weeks ago by hanwenn
Modified:
7 months, 1 week ago
Reviewers:
dak
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Remove unused ly_hash_scm

Patch Set 1 #

Patch Set 2 : dak #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -7 lines) Patch
M lily/include/lily-guile.hh View 1 1 chunk +0 lines, -1 line 0 comments Download
M lily/lily-guile.cc View 1 chunk +0 lines, -6 lines 0 comments Download

Messages

Total messages: 4
dak
Issue title is wrong. Function is ly_scm_hash, not ly_hash_scm. Also its prototype in lily/include/lily-guile.hh should ...
7 months, 2 weeks ago (2020-02-13 12:05:51 UTC) #1
hanwenn
dak
7 months, 2 weeks ago (2020-02-13 12:56:08 UTC) #2
hanwenn
On 2020/02/13 12:05:51, dak wrote: > Issue title is wrong. Function is ly_scm_hash, not ly_hash_scm. ...
7 months, 1 week ago (2020-02-15 17:09:42 UTC) #3
hanwenn
7 months, 1 week ago (2020-02-19 09:36:28 UTC) #4
commit 7cf2dd486822a301ae36928790cfcd81154be449
Author: Han-Wen Nienhuys <hanwen@lilypond.org>
Date:   Thu Feb 13 11:28:14 2020 +0100

    Remove unused ly_scm_hash
    
    https://sourceforge.net/p/testlilyissues/issues/5755
    http://codereview.appspot.com/569340043
Sign in to reply to this message.

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