Neil Conway <[EMAIL PROTECTED]> writes: > I think you're missing the point. Obviously the current code is wrong, > the debate is over the best way to fix it. Jeroen's interpretation of > the spec suggests that merely having libpq_gettext() preserve errno is > not sufficient. I'm not convinced this his interpretation is correct, > but it is a question worth resolving.
(1) The fact that gettext works at all seems to me to be sufficient empirical evidence that it's a working solution. (2) Whether there are sequence points in the function call or not, there most certainly are sequence points inside each called function. The spec allows the functions involved to be called in an unspecified order, but it doesn't allow the compiler to interleave the execution of several functions. (3) Interpretation or not, the approach that Jeroen proposes is unmaintainable; people will not remember to use such a kluge everywhere they'd need to. I'd much rather fix it in one place and do whatever we have to do to keep the compiler from breaking that one place. regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 7: don't forget to increase your free space map settings