reanalyze icon indicating copy to clipboard operation
reanalyze copied to clipboard

Module incorrectly marked dead when contains non-dead type and submodule definitions

Open sim642 opened this issue 3 years ago • 1 comments

Running reanalyze https://github.com/rescript-association/reanalyze/commit/389dd682223201b42c33e5c444e5c79e0805adad on Goblint https://github.com/goblint/analyzer/commit/a544002114324a207df72f9e79eb656c3ebe7b5b revealed the following.

In the terminal output there's a dead module warning:

  Warning Dead Module
  File "./gobview/src/state/gvMessages.ml", line 1, characters 0-0
  +gvMessages is a dead module as all its items are dead.

  Warning Dead Value
  File "./gobview/src/state/gvMessages.ml", lines 22-28, characters 0-18
  +filter_by_line is never used
  <-- line 22
    List.filter pred [@@dead "+filter_by_line"]

Although the only value filter_by_line is indeed dead, it also contains a type t and module Message which are not. Therefore the GvMessages module should not be considered dead overall.

sim642 avatar May 17 '22 08:05 sim642

Not sure if include is treated correctly, for the inner module.

cristianoc avatar May 18 '22 06:05 cristianoc