>>>>> "BT" == Ben Tilly <btilly-Re5JQEeQqe8AvxtiuMwx3w@xxxxxxxxxxxxxxxx> writes:
BT> I would start by believing the error message.
BT> The only string eq in the statement is:
BT> $el_stack[$ind] eq $stack[$ind]
BT> therefore one of those is undefined.
BT> As Uri pointed out, if you put the bounds check before the equality
BT> check you'd likely eliminate the error.
and don't forget to change <= to < if those really were lengths.
i just have a strong feeling that the whole section can be rewritten
much better but i don't have the goal and context i need.
in donald's second post, i saw a bunch more of array indexing and a good
rule in perl is to try to avoid that. parallel arrays are better
represented by arrays of hashes. and then you can use cleaner and faster
for loops instead of c style loops or while's and messing with index
variables (which lead to off by one errors).
uri
--
Uri Guttman ------ uri-X/06uaNR9nFmbZtjAW+qKA@xxxxxxxxxxxxxxxx --------
http://www.stemsystems.com
--Perl Consulting, Stem Development, Systems Architecture, Design and Coding-
Search or Offer Perl Jobs ---------------------------- http://jobs.perl.org
|
