From 49d721741f726cc9f9ddcfd3b260d6190505f476 Mon Sep 17 00:00:00 2001
From: Thomas BESSOU <thomas.bessou@hotmail.fr>
Date: Fri, 12 Jan 2024 01:27:13 +0100
Subject: [PATCH] add more cleanup comment

---
 src/c_api.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/c_api.cpp b/src/c_api.cpp
index 8562815432ef..725323132372 100644
--- a/src/c_api.cpp
+++ b/src/c_api.cpp
@@ -2121,6 +2121,9 @@ int LGBM_BoosterCalcNumPredict(BoosterHandle handle,
 
 // TODO in future versions of LightGBM, public API named around `FastConfig` should be made named around
 // `SingleRowPredictor`, because it is specific to single row prediction, and doesn't actually hold only config.
+// At the same time, one should consider removing the old non-fast single row public API that stores its Predictor
+// in the Booster, because that will enable removing these Predictors from the Booster, and associated initialization
+// code.
 int LGBM_FastConfigFree(FastConfigHandle fastConfig) {
   API_BEGIN();
   delete reinterpret_cast<SingleRowPredictor*>(fastConfig);