fre-hu / mdarray

Multidimensional array for Rust
Apache License 2.0
11 stars 0 forks source link

Panic messages could be more helpful #5

Open grothesque opened 1 month ago

grothesque commented 1 month ago

(For sure the following is luxury, but I'm noting it here nevertheless.)

Currently, panic messages are very terse. Debugging often requires inserting some dbg! invocations. For example mapping.rs has:

assert!(index < self.rank(), "invalid dimension");

I think that it would be helpful if the above was replaced by, for example,

assert!(index < self.rank(), "Expecting a dimension smaller than {}, but got {}.", self.rank(), index);