Menu

#5693 Doc: Corrected doc string for ly:dimension?

Fixed
Enhancement
2020-01-31
2020-01-25
No

ly:dimension? is a predicate - it does not return a number.

https://codereview.appspot.com/547470049

Discussion

  • Anonymous

    Anonymous - 2020-01-26
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Patch: new --> review
     
  • Anonymous

    Anonymous - 2020-01-26

    Passes make, make check and a full make doc.

     
  • Anonymous

    Anonymous - 2020-01-27
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2020-01-27

    Patch on countdown for Jan 29th

     
  • Anonymous

    Anonymous - 2020-01-29
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2020-01-29

    Patch counted down - please push. If you don't have commit access David, can you attach a git formatted patch (from current master) and someone can push it for you? Thank you.

     
  • Anonymous

    Anonymous - 2020-01-31

    David, I cannot seem to apply this patch. Can you rebase it and try again?

    e.g

    james@Zeus:~/lilypond-git$ git am ~/Desktop/0001-Doc-Corrected-doc-string-for-ly-dimension.patch
    Applying: Doc: Corrected doc string for ly:dimension?
    error: patch failed: lily/general-scheme.cc:260
    error: lily/general-scheme.cc: patch does not apply
    Patch failed at 0001 Doc: Corrected doc string for ly:dimension?
    Use 'git am --show-current-patch' to see the failed patch
    When you have resolved this problem, run "git am --continue".
    If you prefer to skip this patch, run "git am --skip" instead.
    To restore the original branch and stop patching, run "git am --abort".
    
     
    • Jonas Hahnfeld

      Jonas Hahnfeld - 2020-01-31

      The patch is already in master, that's why it doesn't apply ;-)

       
  • Jonas Hahnfeld

    Jonas Hahnfeld - 2020-01-31
    • labels: --> Fixed_2_21_0
    • status: Started --> Fixed
    • Patch: push -->
     
  • Jonas Hahnfeld

    Jonas Hahnfeld - 2020-01-31

    in master:

    commit 4e438ed4393302b6a0e3dc77a4541496776beb1f
    Author:     David Stephen Grant <david@davidgrant.no>
    AuthorDate: Sat Jan 25 18:37:29 2020 +0100
    Commit:     David Kastrup <dak@gnu.org>
    CommitDate: Wed Jan 29 16:06:34 2020 +0100
    
        Doc: Corrected doc string for ly:dimension?
    
        ly:dimension? is a predicate - it does not return a number.
    
    diff --git a/lily/general-scheme.cc b/lily/general-scheme.cc
    index d2f3fbdf8b..98a49f5ada 100644
    --- a/lily/general-scheme.cc
    +++ b/lily/general-scheme.cc
    @@ -260,7 +260,7 @@ LY_DEFINE (ly_unit, "ly:unit", 0, 0, 0, (),
     }
    
     LY_DEFINE (ly_dimension_p, "ly:dimension?", 1, 0, 0, (SCM d),
    -           "Return @var{d} as a number.  Used to distinguish length"
    +           "Is @var{d} a dimension?  Used to distinguish length"
                " variables from normal numbers.")
     {
       return scm_number_p (d);
    
     
MongoDB Logo MongoDB