GDScript.@rpc documentation is inconsistent with C# RpcAttribute

[Luminoth]

Your Godot version:

4.2.1

Issue description:

Apologies if this is actually a code bug, I'm not sure where the GDScript code comes from to check on that. I've observed that the GDScript.@rpc documentation says @rpc ( ... transfer_mode="unreliable", ...) but the C# RpcAttribute.cs is defaulting it to MultiplayerPeer.TransferModeEnum.Reliable. Again, I'm not sure if this is an issue with the documentation or an issue with the code.

URL to the documentation page:

https://docs.godotengine.org/en/stable/classes/class_%40gdscript.html#class-gdscript-annotation-rpc

[AThousandShips]

This part is exclusive to GDScript so it doesn't have anything to do with C#, this should be documented specifically for C#, but the GDScript page is unrelated

[raulsntos]

When I implemented the C# attribute the default was reliable, at least according to the PR that added the GDScript annotation (which is what I used as reference):

• [Net] New @rpc annotation, "sync" is no longer part of mode. godot#49882.

So unless the default has changed, it should still be reliable in both GDScript and C# when the transfer mode is unspecified. Looking at the code it looks like the default when unspecified is still reliable:

https://github.com/godotengine/godot/blob/9adb7c7d130c6d4eb0e80b92d6eebd71fac3384d/modules/multiplayer/scene_rpc_interface.cpp#L93

So it could be a bug in the GDScript documentation.

[Luminoth]

Ah ok that's the code I couldn't find. That sure does look like it is the documentation that's wrong on this one.

[nathanfranke]

Tentative fix to https://docs.godotengine.org/en/stable/tutorials/networking/high_level_multiplayer.html#remote-procedure-calls
Edit: Also fixed https://docs.godotengine.org/en/stable/classes/class_@gdscript.html#class-gdscript-annotation-rpc

diff --git a/classes/class_@gdscript.rst b/classes/class_@gdscript.rst
index 6d91468..6e97ca3 100644
--- a/classes/class_@gdscript.rst
+++ b/classes/class_@gdscript.rst
@@ -773,7 +773,7 @@ The order of ``mode``, ``sync`` and ``transfer_mode`` does not matter, but value
     @rpc("any_peer", "unreliable_ordered")
     func fn_update_pos(): pass
     
-    @rpc("authority", "call_remote", "unreliable", 0) # Equivalent to @rpc
+    @rpc("authority", "call_remote", "reliable", 0) # Equivalent to @rpc
     func fn_default(): pass
 
 .. rst-class:: classref-item-separator
diff --git a/tutorials/networking/high_level_multiplayer.rst b/tutorials/networking/high_level_multiplayer.rst
index 5cdff4b..00e418f 100644
--- a/tutorials/networking/high_level_multiplayer.rst
+++ b/tutorials/networking/high_level_multiplayer.rst
@@ -220,7 +220,7 @@ The annotation can take a number of arguments, which have default values. ``@rpc
 
 ::
 
-    @rpc("authority", "call_remote", "unreliable", 0)
+    @rpc("authority", "call_remote", "reliable", 0)
 
 The parameters and their functions are as follows: