[PATCH] remove two compiler warnings for unused variables

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.

Neels Hofmeyr nhofmeyr at sysmocom.de
Tue Apr 12 11:17:34 UTC 2016


On Tue, Apr 12, 2016 at 12:23:52AM +0200, Harald Welte wrote:
> Hi Max,

It was me, not Max :)

> will just mean that we all forget about having to fix it.

Maybe instead of numerous unused variables, we should have a single
#warning then. Every warning hinders my dev cycle. For pretty much every
compilation, I have to parse/browse the list of warnings, *all* the time.

Even a #warning would stare me in the face for every build. There are a
couple of those already. So I'd even prefer a TODO file in / , admitted
that these kind of files tend to go outdated.

Also, having warnings around makes it harder to spot new warnings I've
introduced. I see some warnings scroll by, then assume they're just the
old ones, or have to scroll around to check every time.

I'd prefer a way that I don't have to check for and read the same old
warnings dozens of times every day, while not having the time to
investigate/fix anyway... I unnecessarily lose time on those.

A workaround for me would be to eradicate the warnings on our iu branches
but not commit them to master.

~Neels

-- 
- Neels Hofmeyr <nhofmeyr at sysmocom.de>          http://www.sysmocom.de/
=======================================================================
* sysmocom - systems for mobile communications GmbH
* Alt-Moabit 93
* 10559 Berlin, Germany
* Sitz / Registered office: Berlin, HRB 134158 B
* Geschäftsführer / Managing Directors: Holger Freyther, Harald Welte
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.osmocom.org/pipermail/openbsc/attachments/20160412/9ccae77f/attachment.bin>


More information about the OpenBSC mailing list