LilyIssues / Issues / #5069 Avoid using void * as SCM

#5069 Avoid using void * as SCM

Status: Verified

Owner: David Kastrup

Labels: Fixed_2_19_56 (7)

Needs:

Patch:

Type: Maintainability

Updated: 2017-03-06

Created: 2017-02-14

Creator: David Kastrup

Private: No

Avoid using void * as SCM

This is not really type-safe. Using an SCM * instead is unproblematic.

http://codereview.appspot.com/314500043

Discussion

Anonymous - 2017-02-15

Description has changed:

Diff:

Needs: -->

Patch: new --> review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-02-15

Passes make, make check and a full make doc.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-02-17

Patch: review --> countdown
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-02-17

Patch on countdown for February 20th.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-02-20

Patch: countdown --> push
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-02-20

patch counted down - please push.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

David Kastrup - 2017-02-20

labels: --> Fixed_2_19_56

status: Started --> Fixed

Patch: push -->

Type: Enhancement --> Maintainability
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

David Kastrup - 2017-02-20

Pushed to staging as
commit 3fc70e3e7dfb3c26cb3e5bb9dcabc87cc758c24d
Author: David Kastrup dak@gnu.org
Date: Tue Feb 14 13:41:52 2017 +0100

Issue 5069: Avoid using void * as SCM This is not really type-safe. Using an SCM * instead is unproblematic.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Graham Percival - 2017-03-06

status: Fixed --> Verified
If you would like to refer to this comment somewhere else in this project, copy and paste the following link: