[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [gpsd-dev] cppcheck
|
From: |
Bernd Zeimetz |
|
Subject: |
Re: [gpsd-dev] cppcheck |
|
Date: |
Wed, 30 Jan 2013 18:47:27 +0100 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.12) Gecko/20130116 Icedove/10.0.12 |
On 01/26/2013 09:34 PM, address@hidden wrote:
> driver_nmea2000.c:837: style: Variable 'daddr' is assigned a value that is
> never
> used
[..]
>
> The message is O.K., but should be switched off.
I don't think so. Better just comment out these two lines for later usage. Dead
code is always bad, it either wastes memory or confuses people who want to
uderstand your code...
> I still want to have this two variables and the code in, as i may need it
> later.
add some // in front of these lines and remove them when you need them.
--
Bernd Zeimetz Debian GNU/Linux Developer
http://bzed.de http://www.debian.org
GPG Fingerprint: ECA1 E3F2 8E11 2432 D485 DD95 EB36 171A 6FF9 435F