On 03.09.24 22:56, Jacob Champion wrote:
The parse_strval field could use a better explanation.I actually don't understand the need for this field. AFAICT, this is just used to record whether strval is valid.No, it's meant to track the value of the need_escapes argument to the constructor. I've renamed it and moved the assignment to hopefully make that a little more obvious. WDYT?
Yes, this is clearer. This patch (v28-0001) looks good to me now.
