This is a grey area - the style guide talks about mutators and gives an example strictly as if mutator meant a getter/setter method.  But here, the method is actually using passed args to append/overwrite data in an internal data structure member var.  And the corresponding retrieval method even has some logic for exception throwing.  I'm okay either way, but we should probably evolve a consistent understanding of where we draw the line.