mozilla-services/android-sync — comment on commit 6704de61bd3d6623f59ca332a8ec94e5ebe9beed
File: src/main/java/org/mozilla/gecko/sync/jpake/JPakeClient.java line 439
By rnewman:

Same for this. I have a real dislike of having to split a .method call across two lines; I'd rather have two blocks, and a local variable:

if (iPayload == null ||
iPayload.getObject(Constants.ZKP_KEY_ZKP_A) == null) {
Log.…
…
return;
}
String zkpA = iPayload.getObject(Constants.ZKP_KEY_ZKP_A);
if (zkpA == null ||
!theirSignerId.equals(zkpA.get(Constants.ZKP_KEY_ID))) {
…
return;
}