Skip to content

Comments

SunEdison's fixes for Issues #3, #6, and #7 (and more)#3

Open
emezeske wants to merge 4 commits intogreenenergycorp:masterfrom
emezeske:master
Open

SunEdison's fixes for Issues #3, #6, and #7 (and more)#3
emezeske wants to merge 4 commits intogreenenergycorp:masterfrom
emezeske:master

Conversation

@emezeske
Copy link

@emezeske emezeske commented Dec 9, 2010

Note: Issues below refer to those found on http://code.google.com/p/dnp3/issues/list .

  • Issue SunEdison's fixes for Issues #3, #6, and #7 (and more) #3 - Added support for per-connection TCP keepalive settings for POSIX systems. Adding support for Win32 systems appears to be possible, but I am not a Windows developer so I have not implemented it.
  • Issue #6 - The IOServiceThread no longer silently discards errors (by printing them to cout). It logs them in the typical fashion.
  • Issue #7 - PhysicalLayerAsyncTCPServer::DoOpen() no longer erroneously mucks with the mEndpoint member variable.
  • Misc: Fixed many spelling errors and typos in both comments and log message text.
  • Misc: Fixed a problem with the Boost 1.43 install script that could cause it to fail if wget was redirected when downloading the tarball.

Evan Mezeske added 4 commits December 9, 2010 15:18
IOService::run may be logged (instead of silently discarded).
the TCP settings for a server or client through the stack.  Added
parameters and a Posix implementation for TCP keepalives.
@jadamcrain
Copy link
Collaborator

Awaiting executing CLA from author.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants