Remove reduntant LOG.isTraceEnabled() from logDirtyProperties

diegolovison · gavinking · commit 8d5d5893260d · 2025-05-26T19:22:39.000+02:00
diff --git a/hibernate-core/src/main/java/org/hibernate/event/internal/DefaultFlushEntityEventListener.java b/hibernate-core/src/main/java/org/hibernate/event/internal/DefaultFlushEntityEventListener.java
@@ -663,10 +663,8 @@ private void logDirtyProperties(EntityEntry entry, int[] dirtyProperties) {
 			for ( int i = 0; i < dirtyProperties.length; i++ ) {
 				dirtyPropertyNames[i] = allPropertyNames[dirtyProperties[i]];
 			}
-			if ( LOG.isTraceEnabled() ) {
-				LOG.trace( "Found dirty properties [" + infoString( persister.getEntityName(), entry.getId() )
-							+ "] : "+ Arrays.toString( dirtyPropertyNames ) );
-			}
+			LOG.trace( "Found dirty properties [" + infoString( persister.getEntityName(), entry.getId() )
+						+ "] : "+ Arrays.toString( dirtyPropertyNames ) );
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -663,10 +663,8 @@ private void logDirtyProperties(EntityEntry entry, int[] dirtyProperties) {`
`663`	`663`	`for ( int i = 0; i < dirtyProperties.length; i++ ) {`
`664`	`664`	`dirtyPropertyNames[i] = allPropertyNames[dirtyProperties[i]];`
`665`	`665`	`}`
`666`		`- if ( LOG.isTraceEnabled() ) {`
`667`		`- LOG.trace( "Found dirty properties [" + infoString( persister.getEntityName(), entry.getId() )`
`668`		`- + "] : "+ Arrays.toString( dirtyPropertyNames ) );`
`669`		`- }`
	`666`	`+ LOG.trace( "Found dirty properties [" + infoString( persister.getEntityName(), entry.getId() )`
	`667`	`+ + "] : "+ Arrays.toString( dirtyPropertyNames ) );`
`670`	`668`	`}`
`671`	`669`	`}`
`672`	`670`