Change in vdsm[master]: java_bindings: Add function overloads

smizrahi at redhat.com smizrahi at redhat.com
Thu May 30 16:45:52 UTC 2013


Saggi Mizrahi has posted comments on this change.

Change subject: java_bindings: Add function overloads
......................................................................


Patch Set 3: (1 inline comment)

....................................................
File client/java/vdsm-json-rpc/src/main/java/org/ovirt/vdsm/jsonrpc/JsonRpcClient.java
Line 159: 	public Future<JsonRpcResponse> call(JsonRpcRequest req) {
Line 160: 		final JsonNode json = req.toJson();
Line 161: 		final ByteArrayOutputStream os = new ByteArrayOutputStream();
Line 162: 		try {
Line 163: 			try (JsonGenerator gen = _jfactory.createJsonGenerator(os)) {
This is not the only Java 7 bit I'm using.
There is a lot of diamond type inference.

I hope that by the time we integrate this we move to Java 7
Line 164: 				gen.writeTree(json);
Line 165: 			}
Line 166: 		} catch (IOException e) {
Line 167: 			// Never happens


--
To view, visit http://gerrit.ovirt.org/14823
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I40887a5fd2e3901efdcc9633c5858ae6b2445291
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list