On 14-07-12 09:54, Ferry Huberts wrote:
On 14-07-12 09:09, Alexander Broekhuis wrote:
Hi Ferry,
Thanks for the patches! I'll look into it tomorrow or somewhere next
week.
Some things I notice:
Eclipse path: normally eclipse should pick up the includes itself if
compilation is done from within eclipse. I do have mixed result with this
however, so I'll check if the mentioned includes are any problem.
it should but they were there in the include file
fix:
it should but they were there in the project file already
Added linux/limits.h: Celix doesn't only target Linux as platform, as
such
I don't think those should be added. What are the errors/warnings
resulting
in this needed header? I think it is important to keep the code clean
from
eclipse complains about stuff.
I asked for review because I could not find something like a __linux__
define
I work on olsr as well, and that codebase is very cross platform but
there we have many parts that are OS specific. IMHO you can't avoid that
in cross platform code...
platform specific solutions (at least, all new code, I guess several
older
parts aren't that clean). But everything at least also compiles on OSX,
which definitely doesn't have a linux/limits.h.
If something is going wrong we'll have to check out a platform
independent
solution.
The others I'll need to check out.
tnx
--
Ferry Huberts
