Menu

#5890 slur-configuration: fix size_t conversion warning

Fixed
Enhancement
2020-04-17
2020-04-12
No

Discussion

  • Anonymous

    Anonymous - 2020-04-12
    • Needs: -->
    • Patch: new --> review
     
  • Anonymous

    Anonymous - 2020-04-12

    Passes make, make check and a full make doc

     
  • Anonymous

    Anonymous - 2020-04-15
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2020-04-15

    Patch on countdown for April 17th

     
  • Anonymous

    Anonymous - 2020-04-17
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2020-04-17

    Patch counted down - please push.

     
  • Jonas Hahnfeld

    Jonas Hahnfeld - 2020-04-17
    • labels: --> Fixed_2_21_1
    • status: Started --> Fixed
    • Patch: push -->
     
  • Jonas Hahnfeld

    Jonas Hahnfeld - 2020-04-17
    commit a457b689b062e7dad3ad8e4d2a935638860f453f
    Author:     Han-Wen Nienhuys <hanwen@lilypond.org>
    AuthorDate: Sun Apr 12 12:02:15 2020 +0200
    Commit:     Han-Wen Nienhuys <hanwen@lilypond.org>
    CommitDate: Fri Apr 17 11:59:04 2020 +0200
    
        slur-configuration: fix size_t conversion warning
    
     
  • Dan Eble

    Dan Eble - 2020-04-17

    I'm getting a new warning from slur-scoring today:

    Making lily/out/slur-scoring.o < cc
    /home/user/lilypond-src/lily/slur-scoring.cc: In static member function 'static scm_unused_struct* Slur::calc_control_points(SCM)':
    /home/user/lilypond-src/lily/slur-scoring.cc:390:45: warning: format '%d' expects argument of type 'int', but argument 3 has type 'size_t' {aka 'long unsigned int'} [-Wformat=]
      390 |       total += to_string (" TOTAL=%.2f idx=%d", best->score (), best->index_);
          |                                            ~^                   ~~~~~~~~~~~~
          |                                             |                         |
          |                                             int                       size_t {aka long unsigned int}
          |                                            %ld
    
     
Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.