On Mon, Jan 31, 2022 at 05:46:29PM -0500, Greg Stark wrote: > It looks strange to me that the errorTag struct has a "const char > *tagname" but a "char *tagvalue". I think this is a side effect of the > code and not actually a sensible way to define the struct. Surely they > should both be const?
This review has not been answered, and it was around for two months now. I am marking this patch as returned with feedback. -- Michael
signature.asc
Description: PGP signature