Em qui., 17 de fev. de 2022 às 10:18, Daniel Gustafsson <[email protected]> escreveu:
> > On 17 Feb 2022, at 13:59, Ranier Vilela <[email protected]> wrote: > > > > Em qui., 17 de fev. de 2022 às 09:52, Daniel Gustafsson <[email protected] > <mailto:[email protected]>> escreveu: > > > On 17 Feb 2022, at 13:19, Ranier Vilela <[email protected] <mailto: > [email protected]>> wrote: > > > > > 1. One #ifdef with a mistake, the correct is _WIN32 and not WIN32. > > > > Can you elaborate on this, we are using WIN32 pretty extensively in the > code: > > > > $ git grep "if[n]\{0,1\}def WIN32$"|wc -l > > 511 > > $ git grep "if[n]\{0,1\}def _WIN32$"|wc -l > > 2 > > > > The _WIN32 cases are in the same ECPG testcase. > > > > Why would _WIN32 be correct in this case? > > Sorry, my fault. > > > > I only use _WIN32 and I jumped to conclusions very quickly. > > Question remains though, should src/interfaces/ecpg/test/sql/sqlda.pgc > really > be using WIN32 and not _WIN32, or doesn't it matter? (or does it only > matter > for consistency?) WIN32 and _WIN32 aren't very informative searchterms to > use > for finding more information. > According the StackOverflow: https://stackoverflow.com/questions/662084/whats-the-difference-between-the-win32-and-win32-defines-in-c WIN32 -> SDK _WIN32 -> Compiler (MSVC) WIN32 with MingW (Windows) is 0? For consistency, I think that will use only WIN32, if Postgres uses it extensively. regards, Ranier Vilela
