-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1557/
-----------------------------------------------------------

(Updated 2011-09-02 21:00:31.912847)


Review request for qpid, Kenneth Giusti, michael goulish, and Ted Ross.


Changes
-------

Updated with the initial Windows implementation.


Summary
-------

Provides a new method for providing notification to an interested party when 
new messages are received.

The EventNotifier class can be associated with either a console or agent 
session. The object provides a file descriptor which then becomes readable when 
there are messages to be processed.

This implementation only supports Posix. There is some work necessary to get a 
Windows implementation in place.


Diffs (updated)
-----

  trunk/qpid/cpp/bindings/qmf2/examples/cpp/Makefile.am 1164327 
  trunk/qpid/cpp/bindings/qmf2/examples/cpp/event_driven_list_agents.cpp 
PRE-CREATION 
  trunk/qpid/cpp/include/qmf/AgentSession.h 1164327 
  trunk/qpid/cpp/include/qmf/ConsoleSession.h 1164327 
  trunk/qpid/cpp/include/qmf/PosixEventNotifier.h PRE-CREATION 
  trunk/qpid/cpp/include/qmf/WindowsEventNotifer.h PRE-CREATION 
  trunk/qpid/cpp/src/CMakeLists.txt 1164327 
  trunk/qpid/cpp/src/qmf.mk 1164327 
  trunk/qpid/cpp/src/qmf/AgentSession.cpp 1164327 
  trunk/qpid/cpp/src/qmf/AgentSessionImpl.h PRE-CREATION 
  trunk/qpid/cpp/src/qmf/BaseEventNotifierImpl.h PRE-CREATION 
  trunk/qpid/cpp/src/qmf/BaseEventNotifierImpl.cpp PRE-CREATION 
  trunk/qpid/cpp/src/qmf/ConsoleSession.cpp 1164327 
  trunk/qpid/cpp/src/qmf/ConsoleSessionImpl.h 1164327 
  trunk/qpid/cpp/src/qmf/PosixEventNotifier.cpp PRE-CREATION 
  trunk/qpid/cpp/src/qmf/PosixEventNotifierImpl.h PRE-CREATION 
  trunk/qpid/cpp/src/qmf/PosixEventNotifierImpl.cpp PRE-CREATION 
  trunk/qpid/cpp/src/qmf/WindowsEventNotifier.cpp PRE-CREATION 
  trunk/qpid/cpp/src/qmf/WindowsEventNotifierImpl.h PRE-CREATION 
  trunk/qpid/cpp/src/qmf/WindowsEventNotifierImpl.cpp PRE-CREATION 
  trunk/qpid/cpp/src/tests/EventNotifierTest.cpp PRE-CREATION 
  trunk/qpid/cpp/src/tests/Makefile.am 1164327 

Diff: https://reviews.apache.org/r/1557/diff


Testing
-------

An example agent takes the existing list_agents and uses an EventNotifier to 
respond to incoming messages rather than blocking on the 
ConsoleSession.nextReceiver() API.

A unit test verifies that the file handle provides by the EventNotifier type is 
properly updating on incoming messgaes.


Thanks,

Darryl

Reply via email to