LilyIssues / Issues / #5347 Add and use a Transform data type

Anonymous - 2018-06-16

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 - 2018-06-16

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:

David Kastrup - 2018-06-17

Rebased, reverted to grob rotation for the rotation property.

http://codereview.appspot.com/344970043

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

David Kastrup - 2018-06-17

Needs: -->

Patch: new --> needs_work

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

David Kastrup - 2018-06-17

Oh for **** sake. This patch has the original GC problem back. I'll need another iteration.

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

David Kastrup - 2018-06-17

Refix GC issue

http://codereview.appspot.com/344970043

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

Anonymous - 2018-06-17

Needs: -->

Patch: new --> review

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

Anonymous - 2018-06-17

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 - 2018-06-19

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

Anonymous - 2018-06-19

Patch on countdown for June 22.

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

David Kastrup - 2018-06-21

Rework according to Dan's review

http://codereview.appspot.com/344970043

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

Anonymous - 2018-06-21

Needs: -->

Patch: new --> review

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

Anonymous - 2018-06-21

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 - 2018-06-22

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

Anonymous - 2018-06-22

Patch on countdown for June 25th.

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

Anonymous - 2018-06-25

Patch: countdown --> push
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- David Kastrup - 2018-06-25
  
  "pkx166h" lilypond-pkx@users.sourceforge.net writes:
  
  Patch: countdown --> push
  
  Comment:
  
  This looks like it is OK to push - I see some conversation with Hans
  on Rietveld but it doesn't look like it would stop the patch.
  
  Hans mentioned the introduction of one-shoot classes for disambiguating
  constructor calls without actually proposing any scenario (that I would
  be able to discern) where they would be relevant in connection with this
  patch. To even get to the discussion stage, this would need to be a lot
  more fleshed out in terms of LilyPond code and data.
  
  --
  David Kastrup
  
  If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2018-06-25

This looks like it is OK to push - I see some conversation with Hans on Rietveld but it doesn't look like it would stop the patch.

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

Anonymous - 2018-06-25

labels: --> Fixed_2_21_0

status: Started --> Fixed

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

Issue 5347/1: Add Transform data type
author  David Kastrup <dak@gnu.org> 
    Wed, 13 Jun 2018 15:02:10 +0100 (16:02 +0200)
committer   David Kastrup <dak@gnu.org> 
    Mon, 25 Jun 2018 10:55:16 +0100 (11:55 +0200)
commit  e73dca0ce5800d5d0d6ec29848e5d43d76d342c8

Issue 5347/2: stencil-integral.cc: use Transform
author  David Kastrup <dak@gnu.org> 
    Fri, 15 Jun 2018 15:04:30 +0100 (16:04 +0200)
committer   David Kastrup <dak@gnu.org> 
    Mon, 25 Jun 2018 10:55:16 +0100 (11:55 +0200)
commit  dd6aa28306839da78d7750cab0856acc26ee9b52

Issue 5347/3: stencil-integral: pass SCM as transform values
author  David Kastrup <dak@gnu.org> 
    Sat, 16 Jun 2018 08:29:36 +0100 (09:29 +0200)
committer   David Kastrup <dak@gnu.org> 
    Mon, 25 Jun 2018 10:55:16 +0100 (11:55 +0200)
commit  bcf29b5699930e959ed9671218c93c52b0d0d22e

Add and use a Transform data type

Issue Tracker for LilyPond

Searches

Help

#5347 Add and use a Transform data type

Discussion