Log4cxx
  1. Log4cxx
  2. LOGCXX-177

SocketImpl::accept uses private APR function: apr_wait_for_io_or_timeout

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.10.0
    • Fix Version/s: 0.10.0
    • Component/s: None
    • Labels:
      None
    • Environment:
      Win32 and standard APR 1.2.7

      Description

      When compiling against standard APR 1.2.7 (1.2.8 should be equivalent on this matter), the symbol apr_wait_for_io_or_timeout is unresolved.

      Searching the apr mailing lists it appears that that function is an internal APR function (unix only btw), which only accidentally found its way to an apr_*.h file. See http://marc.info/?l=apr-dev&m=113309765524699&w=2.

      For projects that already use APR for other things it would be desirable to be able to use a standard APR, instead of one patched for the purposes of log4cxx.

        Activity

        Hide
        Curt Arnold added a comment -

        Removed block that called apr_wait_for_io_or_timeout in rev 588797.

        Could not find a trivial way to reproduce the desired behavior with published APR calls, so any call to SocketImpl::accept will block regardless of any calls to setSoTimeout. Open for better solutions, but not my area of expertise. Would appreciate any comments on the resolution.

        Show
        Curt Arnold added a comment - Removed block that called apr_wait_for_io_or_timeout in rev 588797. Could not find a trivial way to reproduce the desired behavior with published APR calls, so any call to SocketImpl::accept will block regardless of any calls to setSoTimeout. Open for better solutions, but not my area of expertise. Would appreciate any comments on the resolution.

          People

          • Assignee:
            Curt Arnold
            Reporter:
            Diego Santa Cruz
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development