Skip to content

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