[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 0/3] Add '-Wunused-module'

From: Ludovic Courtès
Subject: Re: [PATCH 0/3] Add '-Wunused-module'
Date: Mon, 20 Feb 2023 11:51:25 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)

Hey janneke!

Jan Nieuwenhuizen <> skribis:

> Ludovic Courtès writes:
> Hello,
>> The new ‘-Wunused-module’ warning is enabled at ‘-W2’ only.  The main
>> reason for not enabling it at ‘-W1’ is that in the case of modules used
>> at macro-expansion time only, such as (srfi srfi-26), it cannot
>> determine whether a module is definitely unused.  In this case, the
>> compiler reports the module as “possibly unused”, and it is up to the
>> programmer to check that claim.
> [..]
>> Thoughts?
> The `possibly undefined' warnings about the srfi-9 gnu, srfi-26 and
> curried-definition modules are a bit unfortunate, but easy to check.
> It allowed me to trim quite some imports in Dezyne modules.
> Very nice!


> It seems that only re-exporting a (non-macro) variable from an otherwise
> un used module gives a false positive `unused module' warning; it
> doesn't even say `possibly unused module'.  Not a big problem, but can
> anything be done about that?

Oh, I hadn’t thought about that.  It should be possible to fix it, I’ll
take a look.

Thanks for testing!


reply via email to

[Prev in Thread] Current Thread [Next in Thread]