[sling-org-apache-sling-distribution-journal] branch master updated: Refactor bad smells: (#115)

[cschneider]

This is an automated email from the ASF dual-hosted git repository.

cschneider pushed a commit to branch master
in repository 
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-distribution-journal.git


The following commit(s) were added to refs/heads/master by this push:
     new 24c7c00  Refactor bad smells: (#115)
24c7c00 is described below

commit 24c7c00ef470bc92fe5b8b16cb0361874affd2c1
Author: MartinWitt <wittlinger.mar...@gmail.com>
AuthorDate: Tue Mar 14 10:08:44 2023 +0100

    Refactor bad smells: (#115)
    
    - UnnecessaryToStringCall
    The toString() method is not needed in cases the underlying method handles 
the conversion. Also calling toString() on a String is redundant. Removing them 
simplifies the code.
---
 .../org/apache/sling/distribution/journal/bookkeeper/LocalStore.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/src/main/java/org/apache/sling/distribution/journal/bookkeeper/LocalStore.java
 
b/src/main/java/org/apache/sling/distribution/journal/bookkeeper/LocalStore.java
index c3c510a..db9500a 100644
--- 
a/src/main/java/org/apache/sling/distribution/journal/bookkeeper/LocalStore.java
+++ 
b/src/main/java/org/apache/sling/distribution/journal/bookkeeper/LocalStore.java
@@ -88,7 +88,7 @@ public class LocalStore {
         } else {
             serviceResolver.create(parent, storeId, map);
         }
-        LOG.debug(String.format("Stored data %s for storeId %s", 
map.toString(), storeId));
+        LOG.debug(String.format("Stored data %s for storeId %s", map, 
storeId));
     }
 
     public <T> T load(String key, Class<T> clazz) {