Proposal, Re-written API, use virtual

RobTillaart / DEVRANDOM

Arduino library to wrap a random generator in a stream

MIT License

2 stars 1 forks source link

Proposal, Re-written API, use virtual #13

Closed hanschou closed 7 months ago

hanschou commented 10 months ago

This is a major change and all users has to re-write the code using it. Just a proposal.

Look at

DEVRANDOM.h
examples/DEVRANDOM_virtual_compile_time/DEVRANDOM_virtual_compile_time.ino
examples/DEVRANDOM_virtual_run_time/DEVRANDOM_virtual_run_time.ino

TODO - if it should be used one day

Change names of a lot of classes and variables
Test the output result is actually correct
Clean up code
Optimize code

RobTillaart commented 9 months ago

Sorry, I missed this PR, build started

RobTillaart commented 7 months ago

Had a look at this and if Arduino were for pro's it might be the way to go. But given the sheer amount of less fluent programmers I think such a breaking change would cause a lot of new issues. (I get my fair share of "bug reports" when I introduce breaking changes)