Skip to content
This repository has been archived by the owner on Jul 8, 2022. It is now read-only.

Commit

Permalink
Fix bug in event name when pushing an error event to IDL5 clients. (#400
Browse files Browse the repository at this point in the history
)

The events were not sent by ZMQ in this case because the name didn't match the
attribute name the client had susbcribed to.
This bug was triggering some API_MissedEvents error events on the client side.
(Missed some events! Zmq queue has reached HWM?)
  • Loading branch information
bourtemb authored and Ingvord committed Oct 18, 2017
1 parent 3a8e98e commit b719402
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions cppapi/server/zmqeventsupplier.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1707,15 +1707,19 @@ void ZmqEventSupplier::push_event_loop(DeviceImpl *device_impl,EventType event_t
struct SuppliedEventData sent_value;
::memset(&sent_value,0,sizeof(sent_value));

if (*ite == 5)
{
ev_name = EVENT_COMPAT_IDL5 + ev_name;
name_changed = true;
}

if (except == NULL)
{
switch (*ite)
{
case 5:
{
convert_att_event_to_5(attr_value,sent_value,need_free,att);
ev_name = EVENT_COMPAT_IDL5 + ev_name;
name_changed = true;
}
break;

Expand Down

0 comments on commit b719402

Please sign in to comment.