comment on

Actually, it's more like:

No, it isn't.

The OP's code stops at the first occurrence. Yours does not. Hence, you (may) end up removing additional elements, per your own example:

Test code as copied from original nodes:

# shawnhcorey's code:
my @a = ( 1, 2, 3, 4, 2, 5, 7 );
my $last = $#a;
my $i = 0;
while( $i <= $last ){

  if( $a[$i] == 2 ){
    $last --;
    last if $i > $last;
  }

}continue{
  $i ++;
}
$#a = $last;
say "shawnhcorey: @a"; 

# OP's code
@a = ( 1, 2, 3, 4, 2, 5, 7 );
for (@a) {
    if ($_ == 2) {
        pop @a;
        last
    }
}
say "  OP vsespb: @a";
[download]

Output:

shawnhcorey: 1 2 3 4 2
  OP vsespb: 1 2 3 4 2 5
[download]

Anyway, this is moot now that the OP has posted actual code which uses splice to remove an internal element rather than the last element via pop.

In reply to Re^3: foreach argument modification inside loop, followed by return from loop by rjt
in thread foreach argument modification inside loop, followed by return from loop by vsespb

Are you posting in the right place? Check out Where do I post X? to know for sure.
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big> <blockquote> <br /> <dd> <dl> <dt> <em> <font> <h1> <h2> <h3> <h4> <h5> <h6> <hr /> <i> <li> <nbsp> <ol> <p> <small> <strike> <strong> <sub> <sup> <table> <td> <th> <tr> <tt> <u> <ul>
Snippets of code should be wrapped in <code> tags not <pre> tags. In fact, <pre> tags should generally be avoided. If they must be used, extreme care should be taken to ensure that their contents do not have long lines (<70 chars), in order to prevent horizontal scrolling (and possible janitor intervention).
Want more info? How to link or How to display code and escape characters are good places to start.


Do you know where your variables are?
	PerlMonks