Skip to content

Commit c76b303

Browse files
committed
minor refactoring for better readability
Signed-off-by: Ceki Gulcu <ceki@qos.ch>
1 parent 6ccca03 commit c76b303

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

logback-core/src/main/java/ch/qos/logback/core/rolling/SizeAndTimeBasedFNATP.java

+4
Original file line numberDiff line numberDiff line change
@@ -157,6 +157,10 @@ public boolean isTriggeringEvent(File activeFile, final E event) {
157157
return true;
158158
}
159159

160+
return checkSizeBasedTrigger(activeFile, currentTime);
161+
}
162+
163+
private boolean checkSizeBasedTrigger(File activeFile, long currentTime) {
160164
// next check for roll-over based on size
161165
if (invocationGate.isTooSoon(currentTime)) {
162166
return false;

logback-core/src/main/java/ch/qos/logback/core/rolling/TimeBasedRollingPolicy.java

+7
Original file line numberDiff line numberDiff line change
@@ -227,6 +227,13 @@ public String getActiveFileName() {
227227
}
228228
}
229229

230+
/**
231+
* Delegates to the underlying timeBasedFileNamingAndTriggeringPolicy.
232+
*
233+
* @param activeFile A reference to the currently active log file.
234+
* @param event A reference to the current event.
235+
* @return
236+
*/
230237
public boolean isTriggeringEvent(File activeFile, final E event) {
231238
return timeBasedFileNamingAndTriggeringPolicy.isTriggeringEvent(activeFile, event);
232239
}

0 commit comments

Comments
 (0)