> I.e., it seems to me that the loop that iterates over the different
> iterator categories and types should be nested in the one that loops
over
> the tests, and should only be entered when 1 < _rw_argno (test.which,
> StringIds::arg_range)) is true.
  I agree.

> What do you think about the attached patch?
  Looks good.

Farid.