mbarbin / cubzzle

78%

master: 71%

LAST BUILD ON BRANCH force-unused-code-warnings

branch: force-unused-code-warnings

CHANGE BRANCH

x

Reset

• force-unused-code-warnings
• HEAD
• climate
• cmdlang
• commandlang
• commandlang.0.0.1
• main
• master
• pplumbing
• pplumbing-pkgs
• remove-sexp-value
• setup-dune-automagic
• test-dune-pkg-ci
• tests

Build # 69

Build Type

Pull #1

github

Committed by mbarbin

Commit Message

Enable unused code warnings in ppx generated code

This required two small tweaks:

 - 1. Remove the `[@@deriving fields]` as most of what is generated is
   unused. The only used value is the getter `size`.

There is [@deriving fields ~getters] available, to only generate the
getters, but currently I couldn't find a way to make the ppx only
generate the single [size] getter. Thus resorting to add the getter
manually.

- 2. Upon removing the accessor code, another warning was revealed

File "src/box.ml", lines 3-4, characters 4-3:
3 | ....z_shape : Z_shape.t
4 |   ;.............................................
Error (warning 69 [unused-field]): record field z_shape is never read.

Thus I removed it from the record.

Pull Request Pull Request #1: Force unused code warnings in ppx generated code

Run Details

2 of 2 new or added lines in 1 file covered. (100.0%)

244 of 314 relevant lines covered (77.71%)

2169291.51 hits per line