Cache Environment Variables

[hannesdelbeke]

⚡ Cache Environment Variables

[!tip] Avoid Repeated String Lookups

Current:

def on_update(self):
    """Runs every 66ms regardless of activity"""
    if os.getenv("BQT_DISABLE_WRAP") == "1" and ...:
        bqt.manager._blender_window_change(...)
    if os.getenv("BQT_AUTO_ADD", "1") == "1":
        bqt.manager.parent_orphan_widgets(...)

Optimized:

def __init__(self):
    # Cache env var checks at startup
    self._manage_foreground = (
        os.getenv("BQT_DISABLE_WRAP") == "1" and
        os.getenv("BQT_MANAGE_FOREGROUND", "1") == "1"
    )
    self._auto_add = os.getenv("BQT_AUTO_ADD", "1") == "1"

def on_update(self):
    if self._manage_foreground and self.blender_focus_toggled():
        bqt.manager._blender_window_change(self._active_window_hwnd)

    if self._auto_add:
        bqt.manager.parent_orphan_widgets(...)

[hannesdelbeke]

every frame we lookup a env var. we can do it on startup
only con is that if we change env var dynamically, bqt wont update
i dont know why we d need to support dynamic updating though, so this should be non issue