If there were one or more values after the array name in the unshift call, the script would print "1", then consume all available memory before doing anything else.

update: I should have said "you're not prepending anything to the array.

Also, the script might print more things before running out of memory, if you happen to prepend more than one element at a time, and the second element of that set was not "2" or "4". But the script will still blow up.

After reading your post, I am concerned ;-)

I thought that each "unshift" was removing the first entry in the array(ie $array|0|)...where does the "empty set" come in???

 perldoc -f unshift
       unshift ARRAY,LIST
               Does the opposite of a "shift".  Or the opposite
               of a "push", depending on how you look at it.
               Prepends list to the front of the array, and
               returns the new number of elements in the array.

                   unshift(ARGV, '-e') unless $ARGV[0] =~ /^-/;

               Note the LIST is prepended whole, not one element
               at a time, so the prepended elements stay in the
               same order.  Use "reverse" to do the reverse.
[download]

addendum: If you choose to use shift instead of unshift, you will not run out of memory, and you will see only one element ("1") printed out, which is maybe what you were expecting in the first place.

rotfl BUU++

Makeshifts last the longest.

@list = grep { $_ != 2 && $_ != 4 } @list;

my @foo = 1..10;

for (my $i = 0; $i < @foo; $i++)
{
        print "$i ($foo[$i])\n";
        if ($foo[$i] == 5 || $foo[$i] == 6)
        {
                splice(@foo, $i, 1);
                redo;
        }
}
[download]

That's far too complicated a solution for me to leave standing after a code review if I was doing this formally.

If you merely want to avoid processing items based on their value, use a next in a foreach:

for $item (@list) {
  next if $item == 2 or $item == 4;
  ... rest of processing ..
}
[download]

If you want a list that contains all but those items, then use a grep, as you said:

my @newlist = grep { $_ != 2 and $_ != 4 } @list;
[download]

But your for-loop monstrosity is ripe for off-by-one errors, and even if you got everything just right, your maintenance programmer would almost certainly break it.

Extra special hint: walking a list with for-style loops is almost always WRONG. Yes, there are counterexamples, but start with that.

-- Randal L. Schwartz, Perl hacker