[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [patch #9072] Add missing include for lwip/sys.h
From: |
Sylvain Rochet |
Subject: |
[lwip-devel] [patch #9072] Add missing include for lwip/sys.h |
Date: |
Mon, 8 Aug 2016 12:29:41 +0000 (UTC) |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Firefox/45.0 |
Follow-up Comment #5, patch #9072 (project lwip):
Hi Axel,
Well, that's not how it is supposed to work, if your random function actually
live in your sys_arch.c, that's your problem, but this is not how it was
designed.
The current design behind it is to have your random implementation somewhere,
in let's say, random.c/random.h, and then cc.h should looks like this:
#include "random.h"
#define LWIP_RAND() my_random()
Although I agree having it in sys_arch.c is perfectly fine and I would do the
same if I needed a function around my random call :)
Sylvain
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?9072>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Axel Lin, 2016/08/07
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Sylvain Rochet, 2016/08/07
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Axel Lin, 2016/08/07
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Sylvain Rochet, 2016/08/07
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Simon Goldschmidt, 2016/08/08
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Axel Lin, 2016/08/08
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h,
Sylvain Rochet <=
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Sylvain Rochet, 2016/08/08
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Axel Lin, 2016/08/08
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Sylvain Rochet, 2016/08/09
- [lwip-devel] [patch #9072] Add missing include for lwip/sys.h, Axel Lin, 2016/08/09