Li Japin <japinli(at)hotmail(dot)com> writes:> I find the build_regexp_split_result() has redundant codes, we can move it to before the condition check, can we?
Hm, yeah, that looks a bit strange. It was less strange beforec8ea87e4bd950572cba4575e9a62284cebf85ac5, I think.
Pushed with some additional simplification to get rid of therather ugly (IMO) PG_USED_FOR_ASSERTS_ONLY variable.
regards, tom lane