From 21351a058030cc67e50be138ca7bd4f96b448db2 Mon Sep 17 00:00:00 2001
From: nanahi <130121847+na-na-hi@users.noreply.github.com>
Date: Tue, 12 May 2026 08:40:27 -0400
Subject: [PATCH] options/m_property: only get m_option info when needed

Micro optimization.
---
 options/m_property.c | 27 +++++++++++++++++----------
 options/m_property.h |  3 +++
 2 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/options/m_property.c b/options/m_property.c
index 2af6cfaeea..3a67aaac39 100644
--- a/options/m_property.c
+++ b/options/m_property.c
@@ -121,16 +121,23 @@ int m_property_do(struct mp_log *log, const struct m_property *prop_list,
     int r;
 
     struct m_option opt = {0};
-    r = do_action(prop_list, name, M_PROPERTY_GET_TYPE, &opt, ctx);
-    if (r <= 0)
-        return r;
-    mp_assert(opt.type);
-    // Make sure dynamic range from properties with M_PROPERTY_GET_CONSTRICTED_TYPE is applied
-    struct m_option copt = {0};
-    r = do_action(prop_list, name, M_PROPERTY_GET_CONSTRICTED_TYPE, &copt, ctx);
-    if (r == M_PROPERTY_OK) {
-        opt.min = copt.min;
-        opt.max = copt.max;
+    bool get_opt_required[M_PROPERTY_MAX_ACTION] = {
+        [M_PROPERTY_FIXED_LEN_PRINT] = true, [M_PROPERTY_PRINT] = true, [M_PROPERTY_GET_STRING] = true,
+        [M_PROPERTY_MULTIPLY] = true, [M_PROPERTY_SWITCH] = true,
+        [M_PROPERTY_GET_NODE] = true, [M_PROPERTY_SET_NODE] = true,
+    };
+    if (get_opt_required[action]) {
+        r = do_action(prop_list, name, M_PROPERTY_GET_TYPE, &opt, ctx);
+        if (r <= 0)
+            return r;
+        mp_assert(opt.type);
+        // Make sure dynamic range from properties with M_PROPERTY_GET_CONSTRICTED_TYPE is applied
+        struct m_option copt = {0};
+        r = do_action(prop_list, name, M_PROPERTY_GET_CONSTRICTED_TYPE, &copt, ctx);
+        if (r == M_PROPERTY_OK) {
+            opt.min = copt.min;
+            opt.max = copt.max;
+        }
     }
 
     switch (action) {
diff --git a/options/m_property.h b/options/m_property.h
index ba4ba25ad6..5e8351a379 100644
--- a/options/m_property.h
+++ b/options/m_property.h
@@ -97,6 +97,9 @@ enum mp_property_action {
     // Most properties do not implement this.
     //  arg: (ignored)
     M_PROPERTY_DELETE,
+
+    // Total number of actions.
+    M_PROPERTY_MAX_ACTION,
 };
 
 // Argument for M_PROPERTY_SWITCH