[Xastir-dev] Time for a new release?

Curt Mills curt.we7u at gmail.com
Fri Apr 19 13:06:47 PDT 2019


I meant "all warnings" in the first sentence.

On Fri, Apr 19, 2019 at 1:06 PM Curt Mills <curt.we7u at gmail.com> wrote:

> Right. I wasn't talking about turning off warnings, just specific warnings
> that might not be real. I know there's a whole set of specific warnings
> that can be turned on/off 'cuz I was turning on more warnings some years
> ago to make sure the code was as good as we could get it. I stopped doing
> that when the compilers by default started turning on more and more with
> each release.
>
> On Fri, Apr 19, 2019 at 12:37 PM km5vy Tom Russo <russo at bogodyn.org>
> wrote:
>
>> Unfortunately, turning off warnings is a bad idea in this case.  Some of
>> the
>> things it's warning about are really errors, such as misuse of snprintf
>> (extremely common in Xastir), using "sizeof" on a pointer to get the size
>> of
>> the object pointed to by the object (in quite a number of places), and so
>> forth.
>>
>> If we tell GCC to shut up, we miss those errors.
>>
>> There are, of course, a number of these warnings that are just annoyance.
>> For example, it is very common in Xastir for us to use something like
>> strncat
>> or snprintf very deliberatly to cause the copy operation to truncate the
>> source down into the size of the destination.  GCC8 warns us that the
>> operation
>> could cause truncation, because the destination is smaller than the
>> source.
>>
>> But there are many, many more, and some are real errors.
>>
>> On Fri, Apr 19, 2019 at 12:31:43PM -0700, we recorded a bogon-computron
>> collision of the <curt.we7u at gmail.com> flavor, containing:
>> > On Fri, Apr 19, 2019 at 11:30 AM Tom Russo <russo at bogodyn.org> wrote:
>> >
>> > >
>> > > If we want to release again after we stabilize a little, I'd be OK
>> with it,
>> > > but I really doubt I'll be able to get the warning issue in hand
>> anytime
>> > > this
>> > > month.  Unless, perhaps, someone would like to join in the fun and we
>> could
>> > > divide up the work.
>> > >
>> >
>> > I might be able to help a little here and there. I can remember doing
>> the
>> > same thing way back when a couple of times (I'm coming up on 20 years on
>> > this one project!). At more than one point in the project I had it down
>> to
>> > zero errors/warnings. Then another GCC version came out each time.
>> >
>> > One thing we could also perhaps do is change the compile flags to issue
>> > fewer warnings, but of course wouldn't want to turn off ones that are
>> real
>> > and might have implications if we don't correct our code. Don't want to
>> > hide real problems.
>> >
>> > --
>> > Curt, WE7U        http://we7u.wetnet.net
>> http://www.sarguydigital.com
>>
>> --
>> Tom Russo    KM5VY
>> Tijeras, NM
>>
>>  echo "prpv_a'rfg_cnf_har_cvcr" | sed -e 's/_/ /g' | tr [a-m][n-z]
>> [n-z][a-m]
>>
>>
>
> --
> Curt, WE7U        http://we7u.wetnet.net
> http://www.sarguydigital.com
>


-- 
Curt, WE7U        http://we7u.wetnet.net        http://www.sarguydigital.com


More information about the Xastir-dev mailing list