From 8e85c8a402f5a65bce612cc856e6c81828faff4d Mon Sep 17 00:00:00 2001
From: rh-id <58564005+rh-id@users.noreply.github.com>
Date: Sat, 17 Feb 2024 10:47:35 +0700
Subject: [PATCH 1/2] Optimize XDDFChartData.java LOGGER

---
 .../org/apache/poi/xddf/usermodel/chart/XDDFChartData.java   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/chart/XDDFChartData.java b/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/chart/XDDFChartData.java
index c8e92fea643..590f4c95a52 100644
--- a/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/chart/XDDFChartData.java
+++ b/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/chart/XDDFChartData.java
@@ -17,6 +17,8 @@ Licensed to the Apache Software Foundation (ASF) under one or more
 
 package org.apache.poi.xddf.usermodel.chart;
 
+import static org.apache.logging.log4j.util.Unbox.box;
+
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
@@ -170,8 +172,7 @@ public void replaceData(XDDFDataSource<?> category, XDDFNumericalDataSource<? ex
             if (categoryData != null && values != null) {
                 int numOfPoints = category.getPointCount();
                 if (numOfPoints != values.getPointCount()) {
-                    LOGGER.warn("Category and values must have the same point count, but had " +
-                            numOfPoints + " categories and " + values.getPointCount() + " values.");
+                    LOGGER.atWarn().log("Category and values must have the same point count, but had {} categories and {} values.", box(numOfPoints), box(values.getPointCount()));
                 }
             }
             this.categoryData = category;

From 98418114fc62c3314a813da308bbd2b30d7ceb02 Mon Sep 17 00:00:00 2001
From: rh-id <58564005+rh-id@users.noreply.github.com>
Date: Sat, 17 Feb 2024 20:16:19 +0700
Subject: [PATCH 2/2] Optimize XDDFChartData.java LOGGER to remove Unbox

---
 .../org/apache/poi/xddf/usermodel/chart/XDDFChartData.java    | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/chart/XDDFChartData.java b/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/chart/XDDFChartData.java
index 590f4c95a52..b75394c0036 100644
--- a/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/chart/XDDFChartData.java
+++ b/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/chart/XDDFChartData.java
@@ -17,8 +17,6 @@ Licensed to the Apache Software Foundation (ASF) under one or more
 
 package org.apache.poi.xddf.usermodel.chart;
 
-import static org.apache.logging.log4j.util.Unbox.box;
-
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
@@ -172,7 +170,7 @@ public void replaceData(XDDFDataSource<?> category, XDDFNumericalDataSource<? ex
             if (categoryData != null && values != null) {
                 int numOfPoints = category.getPointCount();
                 if (numOfPoints != values.getPointCount()) {
-                    LOGGER.atWarn().log("Category and values must have the same point count, but had {} categories and {} values.", box(numOfPoints), box(values.getPointCount()));
+                    LOGGER.warn("Category and values must have the same point count, but had {} categories and {} values.", numOfPoints, values.getPointCount());
                 }
             }
             this.categoryData = category;