tags 410403 +patch

First, I should warn you that I'm only a somewhat experienced
programmer. I pecked at the code a little bit, and it seems that the
code in question is never actually used. The integer that is being
assigned to is only mentioned twice in the whole code, once to declare,
and once to assign. Never to use.

So, here's a patch to remove the offending code:

----------
diff gpsmap-old.cc gpsmap.cc
2245d2244
<     int thread_status;
2249d2247
<       thread_status = reinterpret_cast<int>(tmp);
----------

Alternatively, if this code actually is used, such as in a future
version, you can change the "int" to a "long". As I understand
"reinterpret_cast", you can change from a pointer to a integral type, as
long as the integral type is large enough to hold the pointer. "long"
would work fine on both 32-bit and 64-bit platforms.

-Brandon


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to