From 5b91040ba6d1f571218acd387b1b3513b98b02d1 Mon Sep 17 00:00:00 2001
From: A Thousand Ships <96648715+AThousandShips@users.noreply.github.com>
Date: Sun, 25 Aug 2024 12:33:20 +0200
Subject: [PATCH] [Doc] Clarify `is_null` methods of `Callable` and `Signal`

---
 doc/classes/Callable.xml | 3 ++-
 doc/classes/Signal.xml   | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/doc/classes/Callable.xml b/doc/classes/Callable.xml
index 05174abb07f..e41174cdd9f 100644
--- a/doc/classes/Callable.xml
+++ b/doc/classes/Callable.xml
@@ -203,7 +203,8 @@
 		<method name="is_null" qualifiers="const">
 			<return type="bool" />
 			<description>
-				Returns [code]true[/code] if this [Callable] has no target to call the method on.
+				Returns [code]true[/code] if this [Callable] has no target to call the method on. Equivalent to [code]callable == Callable()[/code].
+				[b]Note:[/b] This is [i]not[/i] the same as [code]not is_valid()[/code] and using [code]not is_null()[/code] will [i]not[/i] guarantee that this callable can be called. Use [method is_valid] instead.
 			</description>
 		</method>
 		<method name="is_standard" qualifiers="const">
diff --git a/doc/classes/Signal.xml b/doc/classes/Signal.xml
index 07e15d0b237..7d6ff1e9b0c 100644
--- a/doc/classes/Signal.xml
+++ b/doc/classes/Signal.xml
@@ -119,7 +119,7 @@
 		<method name="is_null" qualifiers="const">
 			<return type="bool" />
 			<description>
-				Returns [code]true[/code] if the signal's name does not exist in its object, or the object is not valid.
+				Returns [code]true[/code] if this [Signal] has no object and the signal name is empty. Equivalent to [code]signal == Signal()[/code].
 			</description>
 		</method>
 	</methods>