diff options
-rw-r--r-- | ChangeLog | 25 | ||||
-rw-r--r-- | ChangeLogs/ChangeLog-02a | 25 | ||||
-rw-r--r-- | ChangeLogs/ChangeLog-03a | 25 | ||||
-rw-r--r-- | netsvcs/lib/Client_Logging_Handler.cpp | 70 |
4 files changed, 79 insertions, 66 deletions
diff --git a/ChangeLog b/ChangeLog index a9ab24e48c0..7789d437adc 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,15 +1,28 @@ +Fri Jul 06 18:20:39 2001 Steve Huston <shuston@riverace.com> + + * netsvcs/lib/Client_Logging_Handler.cpp: #if 0'd out a bunch of + ACE_DEBUG statements that aren't appropriate for use in commercial + logging. These can be #if 1'd if there's some doubt while debugging + this stuff, but they aren't useful once debugged, and are + distracting since they're interespersed with application records. + Fri Jul 6 13:40:20 2001 Balachandran Natarajan <bala@cs.wustl.edu> * ace/Log_Msg_UNIX_Syslog.cpp: Fixed a warning in g++. Fri Jul 06 11:39:41 2001 Steve Huston <shuston@riverace.com> - + * ace/INET_Addr.h: Clarified (and improved doxygenation) of constructor and set() methods regarding byte orders. * tests/Max_Default_Port_Test.icc: New file for this test. * tests/tests.icp: Added Max_Default_Port_Test +Fri Jul 6 08:36:20 2001 Chad Elliott <elliott_c@ociweb.com> + + * netsvcs/Makefile: Don't build this if ACE_COMPONENTS does not + contain Other. The netsvcs library requires Naming_Context. + Fri Jul 6 05:44:13 2001 Douglas C. Schmidt <schmidt@tango.doc.wustl.edu> * ace/Makefile (LOGGING_FILES): Added support for @@ -28,16 +41,6 @@ Fri Jul 6 05:44:13 2001 Douglas C. Schmidt <schmidt@tango.doc.wustl.edu> * ace/config-macosx.h: Added a new file courtesy of John Zorko so that things will work on MacOSX. - -Fri Jul 6 08:36:20 2001 Chad Elliott <elliott_c@ociweb.com> - - * netsvcs/Makefile: Don't build this if ACE_COMPONENTS does not - contain Other. The netsvcs library requires Naming_Context. - -Thu Jul 5 23:22:21 2001 Balachandran Natarajan <bala@cs.wustl.edu> - - Fixes that were required for fixing bug 575. Merged in from - branch bug_575_stage_2. Thu Jul 5 23:22:21 2001 Balachandran Natarajan <bala@cs.wustl.edu> diff --git a/ChangeLogs/ChangeLog-02a b/ChangeLogs/ChangeLog-02a index a9ab24e48c0..7789d437adc 100644 --- a/ChangeLogs/ChangeLog-02a +++ b/ChangeLogs/ChangeLog-02a @@ -1,15 +1,28 @@ +Fri Jul 06 18:20:39 2001 Steve Huston <shuston@riverace.com> + + * netsvcs/lib/Client_Logging_Handler.cpp: #if 0'd out a bunch of + ACE_DEBUG statements that aren't appropriate for use in commercial + logging. These can be #if 1'd if there's some doubt while debugging + this stuff, but they aren't useful once debugged, and are + distracting since they're interespersed with application records. + Fri Jul 6 13:40:20 2001 Balachandran Natarajan <bala@cs.wustl.edu> * ace/Log_Msg_UNIX_Syslog.cpp: Fixed a warning in g++. Fri Jul 06 11:39:41 2001 Steve Huston <shuston@riverace.com> - + * ace/INET_Addr.h: Clarified (and improved doxygenation) of constructor and set() methods regarding byte orders. * tests/Max_Default_Port_Test.icc: New file for this test. * tests/tests.icp: Added Max_Default_Port_Test +Fri Jul 6 08:36:20 2001 Chad Elliott <elliott_c@ociweb.com> + + * netsvcs/Makefile: Don't build this if ACE_COMPONENTS does not + contain Other. The netsvcs library requires Naming_Context. + Fri Jul 6 05:44:13 2001 Douglas C. Schmidt <schmidt@tango.doc.wustl.edu> * ace/Makefile (LOGGING_FILES): Added support for @@ -28,16 +41,6 @@ Fri Jul 6 05:44:13 2001 Douglas C. Schmidt <schmidt@tango.doc.wustl.edu> * ace/config-macosx.h: Added a new file courtesy of John Zorko so that things will work on MacOSX. - -Fri Jul 6 08:36:20 2001 Chad Elliott <elliott_c@ociweb.com> - - * netsvcs/Makefile: Don't build this if ACE_COMPONENTS does not - contain Other. The netsvcs library requires Naming_Context. - -Thu Jul 5 23:22:21 2001 Balachandran Natarajan <bala@cs.wustl.edu> - - Fixes that were required for fixing bug 575. Merged in from - branch bug_575_stage_2. Thu Jul 5 23:22:21 2001 Balachandran Natarajan <bala@cs.wustl.edu> diff --git a/ChangeLogs/ChangeLog-03a b/ChangeLogs/ChangeLog-03a index a9ab24e48c0..7789d437adc 100644 --- a/ChangeLogs/ChangeLog-03a +++ b/ChangeLogs/ChangeLog-03a @@ -1,15 +1,28 @@ +Fri Jul 06 18:20:39 2001 Steve Huston <shuston@riverace.com> + + * netsvcs/lib/Client_Logging_Handler.cpp: #if 0'd out a bunch of + ACE_DEBUG statements that aren't appropriate for use in commercial + logging. These can be #if 1'd if there's some doubt while debugging + this stuff, but they aren't useful once debugged, and are + distracting since they're interespersed with application records. + Fri Jul 6 13:40:20 2001 Balachandran Natarajan <bala@cs.wustl.edu> * ace/Log_Msg_UNIX_Syslog.cpp: Fixed a warning in g++. Fri Jul 06 11:39:41 2001 Steve Huston <shuston@riverace.com> - + * ace/INET_Addr.h: Clarified (and improved doxygenation) of constructor and set() methods regarding byte orders. * tests/Max_Default_Port_Test.icc: New file for this test. * tests/tests.icp: Added Max_Default_Port_Test +Fri Jul 6 08:36:20 2001 Chad Elliott <elliott_c@ociweb.com> + + * netsvcs/Makefile: Don't build this if ACE_COMPONENTS does not + contain Other. The netsvcs library requires Naming_Context. + Fri Jul 6 05:44:13 2001 Douglas C. Schmidt <schmidt@tango.doc.wustl.edu> * ace/Makefile (LOGGING_FILES): Added support for @@ -28,16 +41,6 @@ Fri Jul 6 05:44:13 2001 Douglas C. Schmidt <schmidt@tango.doc.wustl.edu> * ace/config-macosx.h: Added a new file courtesy of John Zorko so that things will work on MacOSX. - -Fri Jul 6 08:36:20 2001 Chad Elliott <elliott_c@ociweb.com> - - * netsvcs/Makefile: Don't build this if ACE_COMPONENTS does not - contain Other. The netsvcs library requires Naming_Context. - -Thu Jul 5 23:22:21 2001 Balachandran Natarajan <bala@cs.wustl.edu> - - Fixes that were required for fixing bug 575. Merged in from - branch bug_575_stage_2. Thu Jul 5 23:22:21 2001 Balachandran Natarajan <bala@cs.wustl.edu> diff --git a/netsvcs/lib/Client_Logging_Handler.cpp b/netsvcs/lib/Client_Logging_Handler.cpp index 43cc54b5b01..8fe4c6d5fbf 100644 --- a/netsvcs/lib/Client_Logging_Handler.cpp +++ b/netsvcs/lib/Client_Logging_Handler.cpp @@ -64,10 +64,11 @@ ACE_Client_Logging_Handler::open (void *) ACE_TEXT ("%p\n"), ACE_TEXT ("get_remote_addr")), -1); - +#if 0 ACE_DEBUG ((LM_DEBUG, ACE_TEXT ("connected to client on handle %u\n"), this->peer ().get_handle ())); +#endif /* 0 */ return 0; } @@ -87,9 +88,11 @@ ACE_Client_Logging_Handler::get_handle (void) const int ACE_Client_Logging_Handler::handle_input (ACE_HANDLE handle) { +#if 0 ACE_DEBUG ((LM_DEBUG, ACE_TEXT ("in handle_input, handle = %u\n"), handle)); +#endif /* 0 */ if (handle == this->logging_output_) // We're getting a message from the logging server! @@ -125,8 +128,10 @@ ACE_Client_Logging_Handler::handle_input (ACE_HANDLE handle) ACE_TEXT ("remove_handler")), -1); spipe.close (); +#if 0 ACE_DEBUG ((LM_DEBUG, ACE_TEXT ("client closing down\n"))); +#endif /* 0 */ return 0; } #else @@ -161,8 +166,10 @@ ACE_Client_Logging_Handler::handle_input (ACE_HANDLE handle) this->peer ().close (); else ACE_OS::closesocket (handle); +#if 0 ACE_DEBUG ((LM_DEBUG, ACE_TEXT ("client closing down\n"))); +#endif /* 0 */ return 0; /* NOTREACHED */ @@ -184,8 +191,10 @@ ACE_Client_Logging_Handler::handle_input (ACE_HANDLE handle) 0); ACE_OS::closesocket (handle); +# if 0 ACE_DEBUG ((LM_DEBUG, ACE_TEXT ("client closing down\n"))); +# endif /* 0 */ return 0; } #endif /* ACE_WIN32 */ @@ -206,38 +215,36 @@ ACE_Client_Logging_Handler::handle_input (ACE_HANDLE handle) // seriously wrong, so shutting down the connection is probably // the best solution. if (retrieved != length) - { - ACE_DEBUG ((LM_DEBUG, + { +#if 0 + ACE_DEBUG ((LM_DEBUG, ACE_TEXT ("partial message retrieved, attempting second try...\n"))); +#endif /* 0 */ int remainder = length - retrieved; - - int secondtry = ACE_OS::recv (handle, - ((char *) &log_record) + retrieved, - remainder); - if (secondtry != remainder) - { - ACE_DEBUG ((LM_DEBUG, - ACE_TEXT ("second try failed\n"))); - - if (ACE_Reactor::instance ()->remove_handler - (handle, - ACE_Event_Handler::READ_MASK - | ACE_Event_Handler::EXCEPT_MASK - | ACE_Event_Handler::DONT_CALL) == -1) - ACE_ERROR_RETURN ((LM_ERROR, - ACE_TEXT ("%n: %p\n"), - ACE_TEXT ("remove_handler")), - 0); - - ACE_OS::closesocket (handle); - - ACE_ERROR_RETURN ((LM_ERROR, - ACE_TEXT ("%p\n"), - ACE_TEXT ("recv")), - 0); - } - } + + int secondtry = ACE_OS::recv (handle, + ((char *) &log_record) + retrieved, + remainder); + if (secondtry != remainder) + { + ACE_ERROR ((LM_ERROR, + ACE_TEXT ("%p\n"), + ACE_TEXT ("recv"))); + + if (ACE_Reactor::instance ()->remove_handler + (handle, + ACE_Event_Handler::READ_MASK + | ACE_Event_Handler::EXCEPT_MASK + | ACE_Event_Handler::DONT_CALL) == -1) + ACE_ERROR ((LM_ERROR, + ACE_TEXT ("%n: %p\n"), + ACE_TEXT ("remove_handler"))); + + ACE_OS::closesocket (handle); + return 0; + } + } } #endif /* ACE_HAS_STREAM_PIPES */ @@ -263,9 +270,6 @@ ACE_Client_Logging_Handler::handle_exception (ACE_HANDLE handle) int ACE_Client_Logging_Handler::close (u_long) { - ACE_DEBUG ((LM_DEBUG, - ACE_TEXT ("shutting down!!!\n"))); - if (this->logging_output_ != ACE_STDERR) ACE_OS::closesocket (this->logging_output_); |