Re^2: M3U Playlist Generator

[Anonymous Monk]

Below is my rewritten version which does not depend on File::Find

Why?

Comments welcome.

$inpath can be file ( use -f or -d inestead of -e, since both -f and -d imply -exist )

$inpath could contain glob metacharacters

I'm personally not a fan of interactive commands, exit program if file already exists and overwrite not forced ( Getopt::Long ),

$inpath could contain glob metacharacters

if you don't want to recurse, you could write

use File::Find::Rule;
my @files = find( 'file', 'name', qr/\.mp3$/i, 'maxdepth', 1, 'in', $i
+npath );

FWIW, if you need to do path manipulations, Path::Class::Rule is better than File::Find::Rule;

3 argument open is better than two, "or" is better than "||" since it doesn't require extra parens

open my($fh), '>', $file or die "can't open '>', $file : $! ";

[Soul Singin']

Below is my rewritten version which does not depend on File::Find

Why?

Because I cannot understand the documentation. I'll take a look at your example and keep playing. Thanks!

$inpath can be file ( use -f or -d inestead of -e, since both -f and -d imply -exist )

Good point! -d would be best because the $inpath should be a directory.

$inpath could contain glob metacharacters

Doh! I did not even think of that ... because on my system it will not!

3 argument open is better than two

Old habits are hard to break.

Thanks! Your comments have been very helpful.