Consider using rb_yield_values instead of rb_yield_splat

[ged]

Original report by Anonymous.

---

On line 2178 of pg.c, rb_yield_splat is used in combination with rb_ary_new3 (http://bitbucket.org/ged/ruby-pg/src/eb0d4b1df418/ext/pg.c#cl-2178).

Wouldn't changing line 2178 to be:

rb_yield_values(3, relname, be_pid, extra);

make sense? It should be functionally equivalent and faster. Also, rb_yield_splat isn't supported by rubinius 1.1.1.

[ged]

Original comment by Michael Granger (Bitbucket: ged, GitHub: ged).

---

Apparently so. I chose rb_yield_splat() so it wouldn't be strict about the arity of the block it could yield to, but rb_yield_values() seems to work the same way. I'll change it for the next release.

[ged]

Original comment by Michael Granger (Bitbucket: ged, GitHub: ged).

---

Change rb_yield_splat() to rb_yield_values() for compatibility with Rubinius. (fixes #54)