The only pedantic error was use of variable-length char *. std::string will handle this in C++17 but meanwhile I added a small utility class to provide an RAII version of a char *.

I also took the liberty of fixing a few PEP8 violations and enabling auto PEP8 checking.

I do think the RAII object needs to be adjusted, but it may be best to just drop it in favor of unique_ptr - it turns out you don't even need a custom deleter for that.

Sounds good. Can you please tell me how to compile with -pedantic? I didn't try it.

Easiest thing to do is just use branch tickets/DM-13983 of sconsUtils.

Thank you for the suggestion to use unique_ptr. I did that and it all seems to work. I was able to build using -pedantic.

Please have another look and make sure I used unique_ptr correctly. One question I had is whether it is better to use cstr.get() + i*eltLen (which is what I did) or &cstr[i*eltLen] to access a pointer to an element cstr, a char[] held by a std::unique_ptr

Looks good, thanks for taking care of this! (Now I need to go fix ndarray).

Thank you for the helpful review.