mbed official / EthernetInterface

Important changes to repositories hosted on mbed.com

Mbed hosted mercurial repositories are deprecated and are due to be permanently deleted in July 2026.

To keep a copy of this software download the repository Zip archive or clone locally using Mercurial.

It is also possible to export all your personal repositories from the account settings page.

Dependencies:   lwip-eth Socket lwip lwip-sys

Dependents:   denki-yohou_b Network-RTOS NTPClient_HelloWorld temp_FIAP ... more

David Godfrey

Issue: warning: variable tmp may be used before being set

When doing a Compile All with the latest library (rev 45) the following warning is thrown.

Warning: C4017W: tmp may be used before being set in "EthernetInterface/lwip-eth/arch/TARGET_NXP/lpc_phy_dp83848.c", Line: 317, Col: 6

In function lpc_phy_init tmp is passed to lpc_mii_read on line 332 with the expectation of being set in the function.

The warning could easily be removed by initializing tmp when it is declared....

Line 317

u32_t tmp=0;

3 comments:

Martin Kojtal
02 Feb 2015

Thanks for reporting. I have noticed all issues you reported. Warnings should be all eliminated..

I'll create a new issue for tracking all of them, on github, where mbed SDK development is happening, or you can create an issue there https://github.com/mbedmicro/mbed/issues

David Godfrey
14 Feb 2015

Hi Martin,

Did you get a chance to create that issue on github? I just had a quick look and couldn't find it.

Also, for some reason I didn't get notified that you had commented on this issue!

Martin Kojtal
22 Feb 2015

I just did it, I mgith find time to resolve those ;) https://github.com/mbedmicro/mbed/issues/927