Pierce Thompson

@seeseemelk any chance you could take a look at this?

I'm down for it.

@seeseemelk @TheBusyBiscuit What are your thoughts?

#8334 should resolve this.

That is a fair point. The point of the rename was to reduce the chance of conflicts, and i can see a potential conflict/point of confusion here. It doesn't even...

It's something that would be nice to have done, but not a huge priority. If you're interested in it you can take a shot at it, but it's not required...

Wouldn't it be better to fix the warnings rather than disable them?

I'd rather manually suppress them in code. Disabling all compiler warning is a bad idea.

I had meant linking the setters to the getters with the see tag, not to the impl setters since those aren't really supposed to be public api.

What method are you using to create the snapshot?