3 ms·
This is kind of a bikeshed argument, but I'd prefer if the view was labeled as such. So instead of str_t it would be strview. Rust makes this same mistake IMH
by jandrese 4y ago
This is kind of a bikeshed argument, but I'd prefer if the view was labeled as such. So instead of str_t it would be strview. Rust makes this same mistake IMHO and it causes a lot of confusion for beginners. I would personally call the strbuf_t strstore but that's even more nitpicky.
Naming things is one of the hardest problems in CS.
- mickjc750 4y agoHmm. I think it's a good argument. I'll think on it for a day or 2. I went to some effort to explain the difference between str_t and strbuf_t in the readme, but if someone want's to modify some code using this library one day, it's unlikely they will have seen the readme. In 10 years I'll probably have forgotten how this whole thing works, and that someone may be me.