-
Notifications
You must be signed in to change notification settings - Fork 56
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
Drop "SerialComManager.BAUDRATE baudRate" parameter of SerialComManager.configureComPortData() function #46
Comments
I am unable to understand what is the real problem.
…On Fri, Sep 3, 2021, 4:09 PM powerbroker ***@***.***> wrote:
The "SerialComManager.BAUDRATE baudRate" parameter duplicates the "int
custBaud" parameter. Nothing prevents from invoking
SerialComManager.configureComPortData() with e.g.
BAUDRATE.B115200.getValue() as "custBaud" value.
The first scenario of using configureComPortData() which comes into mind
is:
DATABITS db = DATABITS.DB_8;STOPBITS sb = STOPBITS.SB_1;PARITY p = PARITY.P_NONE;int userTypedBaudRate;
// Some code affecting db, sb, p and userTypedBaudRate here
...configureComPortData(handle, db, sb, p, BAUDRATE.BCUSTOM, userTypedBaudRate);
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#46>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACJVFLDDDYJO7I7GNN6IJQDUACQVNANCNFSM5DLPGXLQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Hi,
Could you please review the attached sample of utilizing SerialPundit
library?
Please focus on the static *private void startCommunication(String ...
args)* method, there are explanation comments.
This is most likely not a problem but usability issue.
I'm implementing a GUI tool for serial IO through RS-485, running a
CDBUS host on PC. There are some rooms in SerialPundit API require extra
coding. The extra coding definitely can be avoided with little changes
in API design.
…On 04.09.2021 19:20, Rishi Gupta wrote:
I am unable to understand what is the real problem.
On Fri, Sep 3, 2021, 4:09 PM powerbroker ***@***.***> wrote:
> The "SerialComManager.BAUDRATE baudRate" parameter duplicates the "int
> custBaud" parameter. Nothing prevents from invoking
> SerialComManager.configureComPortData() with e.g.
> BAUDRATE.B115200.getValue() as "custBaud" value.
>
> The first scenario of using configureComPortData() which comes into mind
> is:
>
> DATABITS db = DATABITS.DB_8;STOPBITS sb = STOPBITS.SB_1;PARITY p =
PARITY.P_NONE;int userTypedBaudRate;
> // Some code affecting db, sb, p and userTypedBaudRate here
> ...configureComPortData(handle, db, sb, p, BAUDRATE.BCUSTOM,
userTypedBaudRate);
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#46>, or unsubscribe
>
<https://github.com/notifications/unsubscribe-auth/ACJVFLDDDYJO7I7GNN6IJQDUACQVNANCNFSM5DLPGXLQ>
> .
> Triage notifications on the go with GitHub Mobile for iOS
>
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
> or Android
>
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
>
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#46 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAPZKQIVQSICB7OVCKN32SLUAJBMFANCNFSM5DLPGXLQ>.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
for some reason web ui fails to attach a file... please use
https://disk.yandex.ru/d/T4tMFgahl01Qfw
…On 04.09.2021 19:20, Rishi Gupta wrote:
I am unable to understand what is the real problem.
On Fri, Sep 3, 2021, 4:09 PM powerbroker ***@***.***> wrote:
> The "SerialComManager.BAUDRATE baudRate" parameter duplicates the "int
> custBaud" parameter. Nothing prevents from invoking
> SerialComManager.configureComPortData() with e.g.
> BAUDRATE.B115200.getValue() as "custBaud" value.
>
> The first scenario of using configureComPortData() which comes into mind
> is:
>
> DATABITS db = DATABITS.DB_8;STOPBITS sb = STOPBITS.SB_1;PARITY p =
PARITY.P_NONE;int userTypedBaudRate;
> // Some code affecting db, sb, p and userTypedBaudRate here
> ...configureComPortData(handle, db, sb, p, BAUDRATE.BCUSTOM,
userTypedBaudRate);
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#46>, or unsubscribe
>
<https://github.com/notifications/unsubscribe-auth/ACJVFLDDDYJO7I7GNN6IJQDUACQVNANCNFSM5DLPGXLQ>
> .
> Triage notifications on the go with GitHub Mobile for iOS
>
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
> or Android
>
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
>
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#46 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAPZKQIVQSICB7OVCKN32SLUAJBMFANCNFSM5DLPGXLQ>.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The "SerialComManager.BAUDRATE baudRate" parameter duplicates the "int custBaud" parameter. Nothing prevents from invoking
SerialComManager.configureComPortData() with e.g. BAUDRATE.B115200.getValue() as "custBaud" value.
The first scenario of using configureComPortData() which comes into mind is:
The text was updated successfully, but these errors were encountered: