Remove `buffer.get` and `buffer.set`

[feross]

These have been deprecated since Apr 27, 2013, and the plan was to remove them in "node v0.13", according to the comments.

buffer.get(index) is superseded by buffer[index].

buffer.set(index, value) is superseded by buffer[index] = value.

I haven't seen these used in real code, ever. They also have never been documented at any point in node's history, according to the git commit history for doc/api/buffer.markdown.

I think it is safe to finally remove these.

Thoughts?

[feross]

Another advantage of removing these is that they'll stop shadowing the underlying TypedArray.prototype.set function.

[jasnell]

@trevnorris

[trevnorris]

Can be removed whenever. It never gets in the way of anything, so I keep forgetting about them. Wouldn't even say it's a major change. Not documented and have been fully deprecated going on 3 years (as @feross mentioned).

[feross]

@trevnorris Okay, PR sent: #4594