in reply to Remove unwanted MP3 tags

Looks good, may I suggest:

1. You are using File::Find::Rule to return a list of all files, then loop over them. That is fine, however, there is a iteration method called match(). Cuts out the need for the list.

2. Parallel::Forkmanager. Since you are looping over files with no ties to each other, why not do it in parallel?

Replies are listed 'Best First'.
Re^2: Remove unwanted MP3 tags
by walto (Pilgrim) on Feb 20, 2014 at 16:07 UTC

    Aye admiral_grinder,

    there is always room for improvements. I adapted the original code, using the method match().

    Thanks for pointing me to a multi threaded approach with Parallel::Forkmanager. I always had problems with threads but I will dig into this module. See what comes up.

[reply]

In Section Cool Uses for Perl