Postgres driver: Use struct in_addr

[jubalh]

Since we later memcopy:

memcpy(&(sadr.sin_addr), (struct in_addr *) &iaddr, sizeof(iaddr));

it is a good idea to to use the in_addr struct instead of the ulong to
make it work among all platforms and architectures.

Apply openSUSE patch: unixODBC-iaddr.patch

[lurcher]

Ok thanks I will make that change though the of driver is historic as a driver example and not recommend to be used. Sent from my Samsung Galaxy smartphone. -------- Original message --------From: Michael Vetter <[email protected]> Date: 22/02/2018 13:09 (GMT+00:00) To: lurcher/unixODBC <[email protected]> Cc: Subscribed <[email protected]> Subject: [lurcher/unixODBC] Postgres driver: Use struct in_addr (#4) Since we later memcopy: memcpy(&(sadr.sin_addr), (struct in_addr *) &iaddr, sizeof(iaddr)); it is a good idea to to use the in_addr struct instead of the ulong to make it work among all platforms and architectures. Apply openSUSE patch: unixODBC-iaddr.patch You can view, comment on, or merge this pull request online at:   #4 Commit Summary Postgres driver: Use struct in_addr File Changes M Drivers/Postgre7.1/socket.c (6) Patch Links: https://github.com/lurcher/unixODBC/pull/4.patch https://github.com/lurcher/unixODBC/pull/4.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread. {"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/lurcher/unixODBC","title":"lurcher/unixODBC","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/lurcher/unixODBC"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Postgres driver: Use struct in_addr (#4)"}],"action":{"name":"View Pull Request","url":"#4"}}}

[jubalh]

Yes even though it is not maintained and only an example I think it's good to have this. I saw several distributions building this driver.

I will make that change

Just pressing 'merge' will be enough :)