Use supplier syntaax for debug log messages
@@ -50,11 +50,11 @@ public class Deconstructor implements ComponentDeconstructor {
}
} else if (component instanceof Provider) {
// TODO Providers should most likely be deconstructed similarly to AbstractComponent
- log.log(LogLevel.DEBUG, "Starting deconstruction of provider " + component);
+ log.log(LogLevel.DEBUG, () -> "Starting deconstruction of provider " + component);
((Provider<?>) component).deconstruct();
- log.log(LogLevel.DEBUG, "Finished deconstruction of provider " + component);
+ log.log(LogLevel.DEBUG, () -> "Finished deconstruction of provider " + component);
} else if (component instanceof SharedResource) {
- log.log(LogLevel.DEBUG, "Releasing container reference to resource " + component);
+ log.log(LogLevel.DEBUG, () -> "Releasing container reference to resource " + component);
// No need to delay release, as jdisc does ref-counting
((SharedResource) component).release();
}
@@ -87,10 +87,10 @@ public class Deconstructor implements ComponentDeconstructor {
@Override
public void run() {
for (var component : components) {
- log.log(LogLevel.DEBUG, "Starting deconstruction of component " + component);
+ log.log(LogLevel.DEBUG, () -> "Starting deconstruction of component " + component);
try {
component.deconstruct();
- log.log(LogLevel.DEBUG, "Finished deconstructing of component " + component);
+ log.log(LogLevel.DEBUG, () -> "Finished deconstructing of component " + component);
} catch (Exception | NoClassDefFoundError e) { // May get class not found due to it being already unloaded
log.log(WARNING, "Exception thrown when deconstructing component " + component, e);
} catch (Error e) {
@@ -110,7 +110,7 @@ public class Deconstructor implements ComponentDeconstructor {
// It should now be safe to uninstall the old bundles.
for (var bundle : bundles) {
try {
- log.log(LogLevel.DEBUG, "Uninstalling bundle " + bundle);
+ log.log(LogLevel.DEBUG, () -> "Uninstalling bundle " + bundle);
bundle.uninstall();
} catch (BundleException e) {
log.log(SEVERE, "Could not uninstall bundle " + bundle);
