[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [lwip-devel] What is xchar() macro?

From: Dirk Ziegelmeier
Subject: Re: [lwip-devel] What is xchar() macro?
Date: Thu, 2 Nov 2017 10:08:50 +0100

The patch is OK for me.

For the other functions: To be correct, they should go to cc.h and be documented in the porting layer. I can do that if desired. But the drawback would be that ctype.h would
​is globally included in many headers of lwIP (because cc.h needs to include it), and given the "complexity" of these functions I'd consider this overkill. It would only be OK to do so if they were used in several places.
​IMHO, the macro names should simply be prepended with lwip_ and be locally defined in the single file that uses them. Opinions, anyone?


On Thu, Nov 2, 2017 at 1:05 AM, Joel Cunningham <address@hidden> wrote:

On 11/01/2017 02:06 PM, address@hidden wrote:
Joel Cunningham wrote:
I think a simple fix would be moving the xchar macro out of that block
and we could guard with #ifndef xchar for whatever bizarre platform
provides it

Or move it out of the block and define it unconditionally with a lwip_ prefix.
Maybe even as a static function. I don't think there's too much to improve here by using a central definition if we can't find one now.

Attached patch implements the conversion to a static lwip_xchar()...and provides some documentation :)


lwip-devel mailing list

reply via email to

[Prev in Thread] Current Thread [Next in Thread]