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
Improve SerialPortManager.getIdentifiers() JavaDoc #1471
Conversation
The current JavaDocs do not clearly state that it does not always return all identifiers. E.g. it will not return any RFC2217 identifiers or undiscovered RXTX port identifiers. So it should not be used to search for an identifier when the serial port name is known. This method is useful for listing available port options in UIs and logging. Signed-off-by: Wouter Born <github@maindrain.net>
* | ||
* @return stream of serial port identifiers | ||
* {@link SerialPortProvider}s may not be able to discover any or all identifiers. | ||
* When the port name is known, the preferred way to get an identifier is by using {@link #getIdentifier(String). |
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.
A missing curly bracket? Isn't it?
* When the port name is known, the preferred way to get an identifier is by using {@link #getIdentifier(String). | |
* When the port name is known, the preferred way to get an identifier is by using {@link #getIdentifier(String)}. |
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.
Well spotted! The curly has returned from its absence. :-)
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.
LGTM. Thanks.
* Improve SerialPortManager.getIdentifiers() JavaDoc The current JavaDocs do not clearly state that it does not always return all identifiers. E.g. it will not return any RFC2217 identifiers or undiscovered RXTX port identifiers. So it should not be used to search for an identifier when the serial port name is known. This method is useful for listing available port options in UIs and logging. Signed-off-by: Wouter Born <github@maindrain.net> GitOrigin-RevId: 583ea02
The current JavaDocs do not clearly state that it does not always return all identifiers.
E.g. it will not return any RFC2217 identifiers or undiscovered RXTX port identifiers.
So it should not be used to search for an identifier when the serial port name is known.
This method is useful for listing available port options in UIs and logging.
Related to: