From Dan Kenigsberg <>:

Dan Kenigsberg has submitted this change and it was merged. ( )

Change subject: RPC: Add missing return statement

RPC: Add missing return statement

In the current code, the function _processEvent()
is not reachable, so this change does not affect anything.
But it will be reachable in subsequent patches.

It is unreachable, because the JsonRpcClient can only
receive messages from a single subscription queue and
this specific queue does not contain events,
only rpc responses

Change-Id: I17a948211ba3aa05af2be69c2a2d366aa26a9e9b
Signed-off-by: Andrej Krejcir <>
M lib/yajsonrpc/
1 file changed, 1 insertion(+), 0 deletions(-)

  Piotr Kliczewski: Looks good to me, approved
  Nir Soffer: Looks good to me, but someone else must approve
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, but someone else must approve
  Andrej Krejcir: Verified

To view, visit
To unsubscribe, visit

Gerrit-MessageType: merged
Gerrit-Change-Id: I17a948211ba3aa05af2be69c2a2d366aa26a9e9b
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Andrej Krejcir <>
Gerrit-Reviewer: Andrej Krejcir <>
Gerrit-Reviewer: Dan Kenigsberg <>
Gerrit-Reviewer: Denis Chaplygin <>
Gerrit-Reviewer: Francesco Romani <>
Gerrit-Reviewer: Irit Goihman <>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <>
Gerrit-Reviewer: Martin Sivák <>
Gerrit-Reviewer: Nir Soffer <>
Gerrit-Reviewer: Phillip Bailey <>
Gerrit-Reviewer: Piotr Kliczewski <>
Gerrit-Reviewer: Roman Mohr <>
Gerrit-Reviewer: Yanir Quinn <>
Gerrit-Reviewer: Yaniv Bronhaim <>
Gerrit-Reviewer: gerrit-hooks <>
vdsm-patches mailing list --
To unsubscribe send an email to

Reply via email to