Re: From MySQL MP3 Database to CD-R

I have one small comment that might make your code more modular, remove:

if ($function =~ /mpg123/) {
[download]

If I'm not mistaken you can pass an argument of undef to system() without it freaking out too much.

-- Dave

Comment on Re: From MySQL MP3 Database to CD-R Select or Download Code

Replies are listed 'Best First'.
(shockme) Re: Re: From MySQL MP3 Database to CD-R by shockme (Chaplain) on Jan 18, 2002 at 07:18 UTC
This is a good point. I hadn't considered passing undef. I did try passing "", which tanked immediately on the mv (complaining that when moving multiple files, the destination must be a directory). I'll give undef a go the next time I burn. If it works, I'll modify the code accordingly. I absolutely hate the if statement in the subroutine. It completely ruins the modularity, so here's to hoping you're correct. If things get any worse, I'll have to ask you to stop helping me.	[reply]

Replies are listed 'Best First'.

(shockme) Re: Re: From MySQL MP3 Database to CD-R
by shockme (Chaplain) on Jan 18, 2002 at 07:18 UTC

undef

I'll give undef a go the next time I burn. If it works, I'll modify the code accordingly. I absolutely hate the if statement in the subroutine. It completely ruins the modularity, so here's to hoping you're correct.

If things get any worse, I'll have to ask you to stop helping me.

[reply]