Properties::parse_from_xml_reader should take Read instead of BufRead

[mainrs]

This makes the method more generic. If a BufReader is needed due to internal implementation, I think it's a better idea to create that inside of the function instead of having the user have to create one themselves.

I'm open to implementing this if you're OK with it.