You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by rm...@apache.org on 2013/10/10 08:47:39 UTC
svn commit: r1530860 -
/commons/sandbox/monitoring/trunk/core/src/main/java/org/apache/commons/monitoring/gauges/Gauge.java
Author: rmannibucau
Date: Thu Oct 10 06:47:39 2013
New Revision: 1530860
URL: http://svn.apache.org/r1530860
Log:
String... is an empty array by default - at least on some jdk - and not null so let handle it
Modified:
commons/sandbox/monitoring/trunk/core/src/main/java/org/apache/commons/monitoring/gauges/Gauge.java
Modified: commons/sandbox/monitoring/trunk/core/src/main/java/org/apache/commons/monitoring/gauges/Gauge.java
URL: http://svn.apache.org/viewvc/commons/sandbox/monitoring/trunk/core/src/main/java/org/apache/commons/monitoring/gauges/Gauge.java?rev=1530860&r1=1530859&r2=1530860&view=diff
==============================================================================
--- commons/sandbox/monitoring/trunk/core/src/main/java/org/apache/commons/monitoring/gauges/Gauge.java (original)
+++ commons/sandbox/monitoring/trunk/core/src/main/java/org/apache/commons/monitoring/gauges/Gauge.java Thu Oct 10 06:47:39 2013
@@ -46,7 +46,7 @@ public interface Gauge {
private void addGaugeIfNecessary(final ClassLoader classLoader, final Gauge g, final boolean excludeParent, final String... prefixes) {
final Class<? extends Gauge> gaugeClass = g.getClass();
if (!excludeParent || gaugeClass.getClassLoader() == classLoader) {
- if (prefixes != null) {
+ if (prefixes != null && prefixes.length > 0) {
boolean found = false;
for (final String p : prefixes) {
if (gaugeClass.getName().startsWith(p.trim())) {
@@ -54,7 +54,9 @@ public interface Gauge {
break;
}
}
- if (!found) return;
+ if (!found) {
+ return;
+ }
}
Repository.INSTANCE.addGauge(g);
gauges.add(g);