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

2023-02-24 Thread Ludovic Courtès
Hello! Jan Nieuwenhuizen skribis: > 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? I fixed that a

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

2023-02-20 Thread Ludovic Courtès
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 >>

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

2023-02-12 Thread Jan Nieuwenhuizen
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

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

2023-02-11 Thread Ludovic Courtès
Hello Guilers! Following a discussion started at the Guix Days and continued online¹, it seems we could benefit from a compiler warning for unused modules, and this is what this patch does. The new ‘-Wunused-module’ warning is enabled at ‘-W2’ only. The main reason for not enabling it at ‘-W1’ i