Parser improvements to handle attributes

I replace "storage class" word with "attribute" in diagnostic messages, because the latter is more general word.
Any reason not to use indexing here instead of pointer arith?  This whole construct is pretty nasty.