Cleanup: use isEmpty instead of size/length

This commit is contained in:
Mathis Chenuet
2015-01-28 20:14:59 +00:00
committed by Laurent Clouet
parent 3497ff7333
commit 4965617784
2 changed files with 3 additions and 3 deletions

View File

@@ -283,7 +283,7 @@ public class Client {
}
// not running but maybe still sending frame
while (this.jobsToValidate.size() > 0) {
while (this.jobsToValidate.isEmpty() == false) {
try {
Thread.sleep(2300); // wait a little bit
}
@@ -403,7 +403,7 @@ public class Client {
String args = "";
if (job_to_reset_ != null) {
args = "?frame=" + job_to_reset_.getFrameNumber() + "&job=" + job_to_reset_.getId() + "&render_time=" + job_to_reset_.getRenderDuration();
if (job_to_reset_.getExtras() != null && job_to_reset_.getExtras().length() > 0) {
if (job_to_reset_.getExtras() != null && job_to_reset_.getExtras().isEmpty() == false) {
args += "&extras=" + job_to_reset_.getExtras();
}
}

View File

@@ -112,7 +112,7 @@ public class Server extends Thread implements HostnameVerifier, X509TrustManager
String args = "";
if (this.client != null && this.client.getRenderingJob() != null) {
args = "?frame=" + this.client.getRenderingJob().getFrameNumber() + "&job=" + this.client.getRenderingJob().getId();
if (this.client.getRenderingJob().getExtras() != null && this.client.getRenderingJob().getExtras().length() > 0) {
if (this.client.getRenderingJob().getExtras() != null && this.client.getRenderingJob().getExtras().isEmpty() == false) {
args += "&extras=" + this.client.getRenderingJob().getExtras();
}
}