impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thomas Tauber-Marshall (Code Review)" <ger...@cloudera.org>
Subject [hs2client-CR] Fix hs2client EXPECT_OK macro.
Date Fri, 13 May 2016 22:25:13 GMT
Thomas Tauber-Marshall has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3068

Change subject: Fix hs2client EXPECT_OK macro.
......................................................................

Fix hs2client EXPECT_OK macro.

The EXPECT_OK macro used in tests references its parameters twice,
leading to duplicate execution. Fix this by updating it to store
the parameter's value to a local veriable so it is only referenced
once.

Change-Id: If44402809dc4e76ec40597caa65e40a202b241c8
---
M src/hs2client/test-util.h
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/hs2client refs/changes/68/3068/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3068
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If44402809dc4e76ec40597caa65e40a202b241c8
Gerrit-PatchSet: 1
Gerrit-Project: hs2client
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <tmarshall@cloudera.com>

Mime
View raw message