search

Robbepop / modular-bitfield

Macro to generate bitfields for structs that allow for modular use of enums.
Apache License 2.0
155 stars 40 forks source link

Ignore unused struct methods #88

Closed vext01 closed 1 year ago

vext01 commented 1 year ago

This fixes #65 in the simplest way possible. It's not ideal, but it's way better than spamming the consumer with compiler warnings.

The first commit makes the tests pass before my change. Seems the compiler errors have changed since those tests were last updated. Note that I haven't checked the test blessing thoroughly. This does also raise the question of if this crate is maintained. Is it maintained?

vext01 commented 1 year ago

I also noticed that cargo fmt doesn't work on this repo:

$ cargo fmt
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
Warning: the `merge_imports` option is deprecated. Use `imports_granularity="Crate"` instead
vext01 commented 1 year ago

This repo is clearly dead. Closing.

Robbepop commented 1 year ago

It would be great to have this issue open for someone to step up and fork the crate if that were to happen.