Menu

#4186 Remove single-C time signature style

Verified
nobody
Enhancement
2015-03-02
2014-11-02
Anonymous
No

Originally created by: *anonymous

Originally created by: nine.fie... (code.google.com)@gmail.com
Originally owned by: nine.fie... (code.google.com)@gmail.com

This style was added to support breve-length measures with a cut-C symbol (Issue 3286).

The single-C style prints
* the cut-C symbol for any time signature with a 2 in the numerator (2/1 answers the feature request)
* the C symbol for any time signature with a 4 in the numerator
* the numerator alone as a number in other cases (like the single-digit style)

Related

Issues: #4308

Discussion

  • Google Importer

    Google Importer - 2014-11-06

    Originally posted by: nine.fie... (code.google.com)@gmail.com

    A note of caution: I am working on converting the C++ code for creating time signatures into Scheme, and when I am done, there is a chance that the people who asked for a style that produces a cut C with a breve-length measure will prefer defining their own print function in Scheme.

    Therefore, it is probably a good idea to defer the documentation work until that question is answered.  The better thing to do might be to remove the single-C feature instead.

     
  • Google Importer

    Google Importer - 2014-12-11

    Originally posted by: nine.fie... (code.google.com)@gmail.com

    The patch for issue 4204 makes it not too difficult for a user to define an idiosyncratic time signature style.  I have requested feedback on 4204 from the person who originally requested that 4/2 be printed as a single cut-C, but I have not received any.  The feedback from others on the single-C style was lukewarm.  Considering all this, I think it would be best to remove the single-C time signature style and let people rely on defining their own styles with markup.

    Summary: Remove single-C time signature style
    Owner: nine.fie...@gmail.com
    Labels: -Type-Documentation Type-Enhancement

     
  • Google Importer

    Google Importer - 2015-01-01

    Originally posted by: nine.fie... (code.google.com)@gmail.com

    Issue 4186: Remove the single-C time signature style

    Remove the single-C time signature style that was added in commit
    [r40b0982cb4f00317c3d50c7f1d4e26fde912f3cd] to address Issue 3286.

    The later commit [r145389636e81237d560aba4d04abcb89fa60cd9c] allows
    the user to define new time signature styles, which is a more
    flexible solution.

    http://codereview.appspot.com/186630043

    Labels: Patch-new
    Status: Started

     
  • Google Importer

    Google Importer - 2015-01-01

    Originally posted by: pkx1... (code.google.com)@gmail.com

    Patchy the autobot says: passes tests.  Includes a full make doc

    Labels: -Patch-new Patch-review

     
  • Google Importer

    Google Importer - 2015-01-04

    Originally posted by: pkx1... (code.google.com)@gmail.com

    Patch on countdown for Jan 7th

    Labels: -Patch-review Patch-countdown

     
  • Google Importer

    Google Importer - 2015-01-07

    Originally posted by: pkx1... (code.google.com)@gmail.com

    Patch counted down - please push

    Labels: -Patch-countdown Patch-push

     
  • Google Importer

    Google Importer - 2015-01-07

    Originally posted by: nine.fie... (code.google.com)@gmail.com

    commit [r82fb2b651b1e38c99342c6163176237e1ea0ec3f]
    Author: Dan Eble <nine.fierce.ballads@gmail.com>
    Date:   Thu Jan 1 15:56:57 2015 -0500

        Issue 4186: Remove the single-C time signature style
       
        Remove the single-C time signature style that was added in commit
        [r40b0982cb4f00317c3d50c7f1d4e26fde912f3cd] to address Issue 3286.
       
        The later commit [r145389636e81237d560aba4d04abcb89fa60cd9c] allows
        the user to define new time signature styles, which is a more
        flexible solution.

    Labels: -Patch-push Fixed_2_19_16
    Status: Fixed

     
  • Google Importer

    Google Importer - 2015-03-02

    Originally posted by: david.na... (code.google.com)@gmail.com

    (No comment was entered for this change.)

    Status: Verified

     
MongoDB Logo MongoDB