Few changes #21

Open
wants to merge 6 commits into
from

Conversation

Projects
None yet
2 participants
@rpt

rpt commented Oct 29, 2012

Here's some commits I made to make the project cleaner and more up-to-date. I hope you will find them useful.

@chancila

This comment has been minimized.

Show comment Hide comment
@chancila

chancila Nov 1, 2012

FYI the trap exits are necessary otherwise terminate won't get called when a link dies or a supervisor requested a shutdown. Seems useful in pgsql_connection at least.

chancila commented Nov 1, 2012

FYI the trap exits are necessary otherwise terminate won't get called when a link dies or a supervisor requested a shutdown. Seems useful in pgsql_connection at least.

@rpt

This comment has been minimized.

Show comment Hide comment
@rpt

rpt Nov 1, 2012

@chancila You're right, thanks. It's only unnecessary in pgsql_sock. Fixed.

rpt commented Nov 1, 2012

@chancila You're right, thanks. It's only unnecessary in pgsql_sock. Fixed.

oferrigni pushed a commit to chef/epgsql that referenced this pull request Feb 10, 2015

Merge pull request #21 from urbanserj/typespecs
More precise specs of functions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment