Skip to content

net: inet_ntop might be broken #1845

@OlegHahm

Description

@OlegHahm

Looking at https://github.com/RIOT-OS/RIOT/blob/master/sys/net/include/inet_ntop.h#L30:

/*
* WARNING: Don't even consider trying to compile this on a system where
* sizeof(int) < 4.  sizeof(int) > 4 is fine; all the world's not a VAX.
*/

I wondered if this could be a problem on MSP430 and ATMega.

I've added data type size comparison to the MCU familiy pages in the wiki (see https://github.com/RIOT-OS/RIOT/wiki/Family%3A-ATmega#data-type-sizes, https://github.com/RIOT-OS/RIOT/wiki/Family%3A-MSP430#data-type-sizes, https://github.com/RIOT-OS/RIOT/wiki/Family%3A-ARM#data-type-sizes, and https://github.com/RIOT-OS/RIOT/wiki/Family:-native#data-type-sizes).

Metadata

Metadata

Labels

Area: networkArea: NetworkingPlatform: AVRPlatform: This PR/issue effects AVR-based platformsPlatform: MSPPlatform: This PR/issue effects MSP-based platformsType: bugThe issue reports a bug / The PR fixes a bug (including spelling errors)Type: questionThe issue poses a question regarding usage of RIOT

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions