heapless icon indicating copy to clipboard operation
heapless copied to clipboard

Add features for queues.

Open reitermarkus opened this issue 6 months ago • 2 comments
trafficstars

Partially addresses https://github.com/rust-embedded/heapless/issues/372.

reitermarkus avatar Apr 27 '25 15:04 reitermarkus

Have you measured how much compile time this saves?

IMO features make sense if they prevent pulling in big dependencies, but in this case they just prevent compiling a few hundred lines of code. In this case the compile time difference is probably negligible, and not worth the dev-experience downgrade due to users having to enable features.

Dirbaio avatar Apr 27 '25 23:04 Dirbaio

No, I didn't measure compile time.

Should we just close https://github.com/rust-embedded/heapless/issues/372 then? The only non-optional dependency we have is hash32, which is pretty tiny as well, with no dependencies itself.

reitermarkus avatar Apr 28 '25 08:04 reitermarkus