Menu

#5904 Optimize get_path_list.

Fixed
2020-04-19
2020-04-13
No

It constructed a list just call memv() on it. Memv does the
hare&tortoise algorithm, so it's also unnecessarily slow.

https://codereview.appspot.com/579570043

Discussion

  • Anonymous

    Anonymous - 2020-04-15
    • Description has changed:

    Diff:

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

    Anonymous - 2020-04-15

    Passes make, make check and a full make doc.

     
  • Anonymous

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

    Anonymous - 2020-04-17

    Patch on countdown for April 19th

    (is it worth a new tracker for the 'I'm going to pass on ... ' issue?)

     
  • Anonymous

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

    Anonymous - 2020-04-19

    Patch counted down - please push.

     
  • Jonas Hahnfeld

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

    Jonas Hahnfeld - 2020-04-19
    commit d48d79b52c28cf9c82d088b2fdf8edbf452d6f94
    Author:     Han-Wen Nienhuys <hanwen@lilypond.org>
    AuthorDate: Mon Apr 13 18:36:46 2020 +0200
    Commit:     Han-Wen Nienhuys <hanwen@lilypond.org>
    CommitDate: Sun Apr 19 09:59:34 2020 +0200
    
        Optimize get_path_list.
    
        It constructed a list just to call memv() on it. Memv does the
        hare&tortoise algorithm, so it's also unnecessarily slow.