Yes, I read something very similar on stackoverflow Charles.
It is probably the way to go, certainly to test out.
As I didn't have the ability right now to test on real hardware,
I just reverted to be safe.
The warnings issue on Jessie and Stretch is as nothing to what is to come
in Buster with gcc 8.xx
There are some pretty intractable warnings due to the coding style in our
legacy code, where buffers are allocated to a defined length
and then a subsequent strncpy() or whatever is fixed to that same
defined length.
The later version of gcc will both warn of potential buffer overflow and of
potential data truncation, even if neither are possible in that instance.
Removing those, if ever deemed worth spending time on, will mean re-writing
a lot of code, possibly with some quite clunky stuff.
On 06/03/19 14:08, Charles Steinkuehler wrote:
On 3/6/2019 5:30 AM, [email protected] wrote:
It essentially comes down to 32 bit and 64 bit differences in data type size.
If you then specify a format size in a printf operation, it will always generate
a warning under one architecture or another.
Assigning with a (void*) cast, will do so too, hence the making of L1 and L2
void * probably
I don't have time to test this in Machinekit, but I've solved these
sort of issues using the intptr_t data type (which changes size based
on the architecture), so something like:
unsigned intptr_t L1, L2;
unsigned long L3;
Another option (since intptr_t isn't guaranteed to be available and
I'm not 100% the pointer casts required for fscanf wouldn't throw a
warning) is to make a union and overlap a pointer with an integer
type, eg:
union foo {
unsigned long long foo_long;
void *foo_ptr;
}
...then use the appropriate type where needed.
--
website: http://www.machinekit.io blog: http://blog.machinekit.io github:
https://github.com/machinekit
---
You received this message because you are subscribed to the Google Groups "Machinekit" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
Visit this group at https://groups.google.com/group/machinekit.
For more options, visit https://groups.google.com/d/optout.
