fleshing out the xml parser a bit