-
-
Notifications
You must be signed in to change notification settings - Fork 33.4k
gh-140505: Correct "parameters" to "arguments" in MultiCall doc #140512
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 3 commits
a190d7b
726c716
6b31ef6
220815b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -15,7 +15,7 @@ | |||||
| -------------- | ||||||
|
|
||||||
| XML-RPC is a Remote Procedure Call method that uses XML passed via HTTP(S) as a | ||||||
| transport. With it, a client can call methods with parameters on a remote | ||||||
| transport. With it, a client can call methods with arguments on a remote | ||||||
| server (the server is named by a URI) and get back structured data. This module | ||||||
| supports writing XML-RPC client code; it handles all the details of translating | ||||||
| between conformable Python objects and XML on the wire. | ||||||
|
|
@@ -195,7 +195,7 @@ grouped under the reserved :attr:`~ServerProxy.system` attribute: | |||||
|
|
||||||
| .. method:: ServerProxy.system.methodSignature(name) | ||||||
|
|
||||||
| This method takes one parameter, the name of a method implemented by the XML-RPC | ||||||
| This method takes one argument, the name of a method implemented by the XML-RPC | ||||||
| server. It returns an array of possible signatures for this method. A signature | ||||||
| is an array of types. The first of these types is the return type of the method, | ||||||
| the rest are parameters. | ||||||
|
|
@@ -204,7 +204,7 @@ grouped under the reserved :attr:`~ServerProxy.system` attribute: | |||||
| a list of signatures rather than a singleton. | ||||||
|
|
||||||
| Signatures themselves are restricted to the top level parameters expected by a | ||||||
| method. For instance if a method expects one array of structs as a parameter, | ||||||
| method. For instance if a method expects one array of structs as a argument, | ||||||
|
||||||
| method. For instance if a method expects one array of structs as a argument, | |
| method. For instance if a method expects one array of structs as an argument, |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is applied to the return type too, I suggest to use this in the example.
| three integers and returns a string, its signature is "string, int, int, int". | |
| three integers and returns an array of strings, its signature is "array, int, int, int". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’ve made the changes you suggested. The section now makes it clear that a signature only shows the top-level XML-RPC type (array or structure) for arguments and return values. Added a couple of examples to make it easier to understand
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would write something like "Note that a signature does not give details of a complex type; it just gives the basic XML-RPC type -- array or structure." (see https://xmlrpc-c.sourceforge.io/introspection.html).
This is applied not only to the types of parameters, but to the return type too.