Although I expect that it should be a double quoted string unless eval EXPR is somehow used later. $sids[$index] looks more Perl-ish than bourne-ish.