Boost C++ Libraries: Ticket #5688: boost::asio::async_read_until with boost::regex compilation error

pexu@… — Tue, 19 Jul 2011 12:43:48 GMT

I hit the same problem.

I believe that the error locates at asio/impl/read_until.hpp:859 where typename RegEx template parameter is declared but not used as the the member function definition says `const boost::regex& expr'.

I guess the it should say `const RegEx & expr' etc.

An excerpt from asio/impl/read_until.hpp:859:

  template <typename AsyncReadStream, typename Allocator,
      typename RegEx, typename ReadHandler>
  inline read_until_expr_op<AsyncReadStream, Allocator, RegEx, ReadHandler>
  make_read_until_expr_op(AsyncReadStream& s,
      boost::asio::basic_streambuf<Allocator>& b,
      const boost::regex& expr, ReadHandler handler)
  {
    return read_until_expr_op<AsyncReadStream, Allocator, RegEx, ReadHandler>(
        s, b, expr, handler);
  }

If I'm missing something deeper here, the documentation should be perhaps updated to reflect changes (didn't spot anything related to async_read_until).

anonymous — Fri, 05 Aug 2011 20:14:46 GMT

I just hit this too. It was a bit painful to track down and it wasn't till *after* I figured out it was the regex that I had enough search terms to find this ticket.

anonymous — Wed, 10 Aug 2011 15:37:41 GMT

This is a simple fix. Replace the function signature from:

make_read_until_expr_op(AsyncReadStream& s,

boost::asio::basic_streambuf<Allocator>& b, const boost::regex& expr, ReadHandler handler)

to:

make_read_until_expr_op(AsyncReadStream& s,

boost::asio::basic_streambuf<Allocator>& b, RegEx expr, ReadHandler handler)

anonymous — Wed, 07 Sep 2011 20:56:04 GMT

I, also, just hit this. Unfortunately, the fix above does not change the compiler error for me (although I certainly seems like it should!).

chris_kohlhoff — Sat, 08 Oct 2011 21:03:24 GMT

(In [74814]) Fix compile error in regex overload of async_read_until.hpp. Refs #5688

status changed; resolution set

chris_kohlhoff — Sun, 09 Oct 2011 22:00:09 GMT

status new → closed
resolution → fixed

(In [74863]) Merge from trunk...

Fix compile error in regex overload of async_read_until.hpp. Fixes #5688

Explicitly specify the signal() function from the global namespace. Fixes #5722

Don't read the clock unless the heap is non-empty.

Change the SSL buffers sizes so that they're large enough to hold a complete TLS record. Fixes #5854

Make sure the synchronous null_buffers operations obey the user's non_blocking setting. Fixes #5756

Set size of select fd_set at runtime when using Windows.

Disable warning due to const qualifier being applied to function type.

Fix crash due to gcc_x86_fenced_block that shows up when using the Intel C++ compiler. Fixes #5763

Specialise operations for buffer sequences that are arrays of exactly two buffers.

Initialise all OpenSSL algorithms.

Fix error mapping when session is gracefully shut down.

Various performance improvements:

Split the task_io_service's run and poll code.

Use thread-local operation queues in single-threaded use cases (i.e. concurrency_hint is 1) to eliminate a lock/unlock pair.

Only fence block exit when a handler is being run directly out of the io_service.

Prefer x86 mfence-based fenced block when available.

Use a plain ol' long for the atomic_count when all thread support is disabled.

Allow some epoll_reactor speculative operations to be performed without holding the lock.

Improve locality of reference by performing an epoll_reactor's I/O operation immediately before the corresponding handler is called. This also improves scalability across CPUs when multiple threads are running the io_service.

Pass same error_code variable through to each operation's complete() function.

Optimise creation of and access to the io_service implementation.

Remove unused state in HTTP server examples.

Add latency test programs.