Conversation
|
Thanks for your contribution. The original code was correct. After your modification, there will be an extra space at the end. Please upload a screenshot to show your issue. |
|
Here is the screenshot of the loopback test case. I'm using v0.5.34, I believe it's the latest. 
|
|
Sending the data twice? |
| for(int i = 0; i < nLen; i++) | ||
| { | ||
| if(i) | ||
| szOut += " "; |
There was a problem hiding this comment.
Are you sure? There will be two spaces between the bytes.
There was a problem hiding this comment.
Change to
if (i && m_Stats.GetTotalReceives())
|
Yes, twice. So I think the extra space at the end is necessary to separate two sets of data. |
|
Please test e78c24e |
|
I can't test it recently, but your modification looks good to me. If you think it fixed my issue, you can close my PR. |
|
Actually, it got worse. I think the condition you modified should be 
|
|
I can't test it right now, I'll make the changes after the holiday. |
|
Change to or. |
|
It's working properly now. |
2 participants
I think there should be a space after each byte; otherwise, the output from the two sets of data will run together, making it difficult to read.