microsoft / tiny-calc

Tiny expression evaluator
MIT License
39 stars 9 forks source link

Nano: Update IMatrixConsumer interface #51

Closed DLehenbauer closed 4 years ago

DLehenbauer commented 4 years ago

Remove values: T[] from row/col insertion/removal notifications. Could bring this back as an optional arg, but currently insert/remove ops always insert empty row/cols.

Rename 'cellsReplaced' -> cellsChanged to match other IConsumer interfaces.

Make values: T[] optional on cellsChanged. This could be useful for bulk set operations, but shouldn't be required.