Skip to content
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

Update PangolinMQTT.cpp #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sticilface
Copy link

Comment out Serial object print. ? change it to PANGO_PRINT
I note #8

Is there a reason that the

void PANGO::dumphex(uint8_t* mem, size_t len,uint8_t W) {

Could also not be changed to

void PANGO::dumphex(Print & output, uint8_t* mem, size_t len,uint8_t W) 

and forgo any dependance on Serial all together. I also do not have a Serial Object for most of my projects and output debugging over a telnet interface. I also use the Serial interface for communication with other devices, ie PJON so printing to it corrupts that communication. I can of course comment out the lines and roll my own hex dump but that means changing it every pull.

Comment out Serial object print.  ? change it to PANGO_PRINT
I note philbowles#8

Is there a reason that the https://github.com/philbowles/PangolinMQTT/blob/300d2b38689f66db6344da2488ea429116555ef5/src/PANGO.cpp#L204

Could not be changed to 

```cpp
void PANGO::dumphex(Print & output, uint8_t* mem, size_t len,uint8_t W) 
```

and forgo any dependance on `Serial` all together.  I also do not have a Serial Object for most of my projects and output debugging over a telnet interface.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant