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

#if DHCPS_DEBUG is missing in dhcpserver.c (IDFGH-557) #2924

Closed
luc-github opened this issue Jan 7, 2019 · 4 comments
Closed

#if DHCPS_DEBUG is missing in dhcpserver.c (IDFGH-557) #2924

luc-github opened this issue Jan 7, 2019 · 4 comments

Comments

@luc-github
Copy link
Contributor

#if DHCPS_DEBUG is missing
So Debug message is alway appearing on serial output

DHCPS_LOG("dhcps: send_offer>>udp_sendto result %x\n", SendOffer_err_t);

@Alvin1Zhang Alvin1Zhang changed the title #if DHCPS_DEBUG is missing in dhcpserver.c [TW#28364] #if DHCPS_DEBUG is missing in dhcpserver.c Jan 13, 2019
@zhangyanjiaoesp
Copy link
Collaborator

Hi @luc-github,
We already know the problem, and we are planing to add it in v4.0. Thanks .

@luc-github
Copy link
Contributor Author

@zhangyanjiaoesp 3.1.3 esp-idf is just released, may I know when V4.0 is planned ?

Is there a reason to wait for a major release to add the define line?

#if DHCPS_DEBUG
DHCPS_LOG("dhcps: send_offer>>udp_sendto result %x\n", SendOffer_err_t);
#endif

@projectgus projectgus changed the title [TW#28364] #if DHCPS_DEBUG is missing in dhcpserver.c #if DHCPS_DEBUG is missing in dhcpserver.c (IDFGH-557) Mar 12, 2019
@igrr igrr closed this as completed in 27cc0d1 Mar 13, 2019
@luc-github
Copy link
Contributor Author

@igrr Thank you
I think there is another one looking at code

DHCPS_LOG("dhcps: send_nak>>udp_sendto result %x\n", SendNak_err_t);

but never saw it in output, so not sure it can be a problem

@zhangyanjiaoesp
Copy link
Collaborator

@luc-github I'm sorry that I didn't check all the "DEBUG_LOG" code, this place need to be changed also. Thanks for your reminding.

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

No branches or pull requests

3 participants