From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has submitted this change and it was merged. ( 
https://gerrit.ovirt.org/79328 )

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 <akrej...@redhat.com>
---
M lib/yajsonrpc/__init__.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  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 https://gerrit.ovirt.org/79328
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I17a948211ba3aa05af2be69c2a2d366aa26a9e9b
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Andrej Krejcir <akrej...@redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrej...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchap...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Irit Goihman <igoih...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jto...@redhat.com>
Gerrit-Reviewer: Martin Sivák <msi...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbai...@redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Gerrit-Reviewer: Roman Mohr <rm...@redhat.com>
Gerrit-Reviewer: Yanir Quinn <yqu...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to