Menu

#2 Patch for delete argument

closed-accepted
nobody
None
5
2003-09-24
2003-09-22
Dave O
No

I noticed that a trailing space in the argument to the
-d flag would result in deleting track 0 because atoi()
returns 0 on failure.

Rather than simply patch this, I decided that whole
function could use a rewrite.

This updated version can take multiple comma or space
seperated track numbers or pairs and will not
erroneously delete an unspecified track.

All of the following are valid and behave as expected:

"1,2,3"
"1-5"
"1,3,6-9"
"2 4 6 8-10"
"2,4,6 9-13"

Discussion

  • Dave O

    Dave O - 2003-09-22

    Patch to fix delete argument

     
  • Nathan Hjelm

    Nathan Hjelm - 2003-09-24
    • status: open --> closed
     
  • Nathan Hjelm

    Nathan Hjelm - 2003-09-24

    Logged In: YES
    user_id=203099

    Applied (v1.4.2b)

     
  • Nathan Hjelm

    Nathan Hjelm - 2003-09-24
    • status: closed --> closed-accepted
     

Log in to post a comment.