From: Iwata, Aya/岩田 彩 <iwata....@fujitsu.com> > I update patch to v18. It has been fixed in response to Tsunakawa san's > review.
(52) + of tracing. If (<literal>flags</literal> contains <literal>PQTRACE_SUPPRESS_TIMESTAMPS</literal>), () can be removed? (53) + int inLogging; /* next byte of logging */ I understood this variable contains a position or offset from some place. Then, isn't it better to call it LogPos, LogOffset, or LogCursor? I don't think you need to prepend "In" as in InCursor, because you don't have to distinguish between input and output unlike InCursor in PGconn. (54) Can't you restrict the use of the above InLogging in fe-logging.c? How about making InLogging mean the offset from InStart, i.e. the offset from the start of a message? Regards Takayuki Tsunakawa
