Michael Paquier <mich...@paquier.xyz> writes: > I find the choice to keep err_msg in next_token() a bit confusing in > next_field_expand(). If no errors are possible, why not just get rid > of it?
Yeah, I dithered about that. I felt like removing it only to put it back later would be silly, but then again maybe there won't ever be a need to put it back. I'm OK with removing it if there's not objections. regards, tom lane