|Subject:||Re: [Paparazzi-devel] Source code analysis of paparazzi|
|Date:||Sun, 30 Nov 2014 11:01:46 +0100|
I agree to Felix.
@Matthieu possibly you should share the chosen compiler settings that you use so we can review them. Bear in mind this is not a 64bits PC so int and const numbers manifest in 32 bits not in 64 bits.
Date: Fri, 28 Nov 2014 21:35:10 +0100
Subject: Re: [Paparazzi-devel] Source code analysis of paparazzi
Seems like your parser adds the (long) casts as that is nowhere in our code:
Hi Matthieu,I can't reproduce the issue.
#define INT32_RAD_OF_DEG(_deg) (int32_t)(((int64_t)(_deg) * 14964008)/857374503)
So this looks good to me as you should be able to pass up to 61637 degrees to that function...
Maximum you can pass before intermediate multiplication overflows:
pow(2,63) / 14964008 = 616370429423 in deg*1e7= 61637 in deg
On Wed, Nov 26, 2014 at 4:53 PM, Matthieu Lemerre <address@hidden> wrote:
_______________________________________________ Paparazzi-devel mailing list address@hidden https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
|[Prev in Thread]||Current Thread||[Next in Thread]|