reorganize conf functions

Signed-off-by: Christian López Barrón <[email protected]>

docs/source/conf.py

@@ -248,17 +248,10 @@
 }
 
 
-def _handle_missing_usd_reference(app, env, node, contnode):
-    """Handle missing references by redirecting to a custom URL."""
-    from docutils import nodes
-
-    target = node['reftarget']
-    if not target.startswith('pxr.'):
-        return None
-
-    reftitle, refuri = _get_url_for_target(app, target)
-    node = nodes.reference('', contnode.astext(), internal=False, refuri=refuri, reftitle=reftitle)
-    return node
+@functools.cache
+def _get_doxylink_part(text):
+    __, __, part = _doxylink_ext.split_explicit_title(text)
+    return _doxylink_ext.utils.unescape(part)
 
 
 @functools.cache
@@ -268,13 +261,6 @@ def _get_usd_ref_tooltip(app):
     return f"(in {proj} v{version})"
 
 
-@functools.cache
-def _doxylink_split_explicit_title(text):
-    has_explicit_title, title, part = _doxylink_ext.split_explicit_title(text)
-    part = _doxylink_ext.utils.unescape(part)
-    return has_explicit_title, title, part
-
-
 @functools.cache
 def _get_url_for_target(app, target):
     pxr_obj_namespace = target.removeprefix('pxr.').replace(".", "")
@@ -285,13 +271,26 @@ def _get_url_for_target(app, target):
         "Usd_Term": "primFlags.h",
         "Usd_PrimFlagsConjunction": "primFlags.h",
     }.get(pxr_obj_namespace, pxr_obj_namespace)
-    __, __, part = _doxylink_split_explicit_title(pxr_obj_namespace)
+    part = _get_doxylink_part(pxr_obj_namespace)
     url = app.env.doxylink_cache[_USD_DOXYGEN_CACHE_NAME]['mapping'][part]
     full_url = _doxylink_ext.join(_USD_DOXYGEN_ROOT_DIR, url.file)
     reftitle = _get_usd_ref_tooltip(app)
     return part + " " + reftitle, full_url
 
 
+def _handle_missing_usd_reference(app, env, node, contnode):
+    """Handle missing references by redirecting to a custom URL."""
+    from docutils import nodes
+
+    target = node['reftarget']
+    if not target.startswith('pxr.'):
+        return None
+
+    reftitle, refuri = _get_url_for_target(app, target)
+    node = nodes.reference('', contnode.astext(), internal=False, refuri=refuri, reftitle=reftitle)
+    return node
+
+
 def _grill_process_signature(app, what, name, obj, options, signature, return_annotation):
     if name == "grill.usd.iprims":  # Target the specific function
         # Prim predicates don't have a __repr__, which creates malformed signature formatting (no new lines, no links)
@@ -304,14 +303,15 @@ def _grill_process_signature(app, what, name, obj, options, signature, return_an
 
 
 def _create_doxylink_role_with_title(app, *args, **kwargs):
+    # TODO: find a less hacky way of achieving this (aiming for consistency with intersphinx titles on hovered links)
     doxylink_role = _doxylink_create_role(app, *args, **kwargs)
     intersphinx_title = _get_usd_ref_tooltip(app)
 
     def _find_doxygen_link_with_title(name, rawtext, text, *patched_args, **patched_kwargs):
         inner_result = doxylink_role(name, rawtext, text, *patched_args, **patched_kwargs)
         node = inner_result[0][0]
 
-        __, __, part = _doxylink_split_explicit_title(text)
+        part = _get_doxylink_part(text)
 
         node.attributes['reftitle'] = part + " " + intersphinx_title
         return inner_result