Fix the warning MSB8028: The intermediate directory (..\x86_Unicode_Release\) contains files shared from another project (pgxalib.vcxproj).  This can lead to incorrect clean and rebuild behavior.

It's safe to add this change in addition to the previous commit.
d0c993d3f6f0a4496a20486f2ec19b14591baba7
Author: U-Lenovo-PC\hiroshi <hiroshi@Lenovo-PC.(none)>
Date:   Sat Jun 14 21:53:57 2014 +0900

    Fix potentially uninitialized local pointer access which causes
    fatal compilation errors on vs2012/2013 reported by Craig Ringer.

Fix potentially uninitialized local pointer access which causes
fatal compilation errors on vs2012/2013 reported by Craig Ringer.

Add top-level Windows makefile targets for building installers.

You can now do "nmake /f win64.mak world" to build both x86 and x64
versions of the installers in one command.

Make sure that the psqlodbc ANSI driver is ODBC3.5.

Changes by a patch and a request from Craig Ringer <craig@2ndquadrant.com>.
Applied a patch [ODBC] PATCH: WiX has moved.
Register 64bit XA DLL on installation per discussion with him.

Fix a PowerShell syntax error.
Also handle the case when GSSBINDIR parameter is given as an empty string in psqodbcm_cpu.wxs.

Support running regression tests on Windows.

Some tests fail, and numeric test crashes for me, but at least it's
relatively easy to run the suite on Windows now.

Michael Paquier and me

Fix ODBC function escape for SPACE function.

Also add a regression test case for SPACE and a few other ODBC function
escapes.

Silence compiler warnings from regression tests.

Three categories of warnings:

1. Missing return at end of main() function
2. Unused variables (copy-pasted from another test)
3. Data type mismatch in printfs, e.g. passing SQLULEN param to %d

Oops, forgot to git add some last-minute changes to the new regression test.

Rewrite the conversion functions between strings and SQL_NUMERIC_STRUCT.

Aside from making the functions more readable and faster, this fixes
a number of bugs:

1. When converting a large number from decimal to binary, the least
significant byte might be wrong (reported by Walter Couto)

2. In binary->decimal conversion, a number with small precision but large
scale caused a "Floating point exception".

3. In binary->decimal conversion, building the final string was wrong, picking
up "digits" from outside the reserved stack space, if scale was larger than 40

4. The param_string buffer allocated for the result of binary->decimal
conversion was too small.

Also add a test cases for all of these bugs.

Minor code cleanup.

The main change is to move the responsibility of adding the "length" header
to the QueryBuild from ResolveOneParam to its caller, when constructing the
parameters to send over to the server in a Bind message. ResolveOneParam
is complicated enough without that, so it seems more clear this way.

The rest is just fixes to the macros to make them deal with a "&qb" argument,
by adding parens around the macro argument. And while at it, clean up the
whitespace in them.

Misc whitespace and comment fixes. Remove dead, commented-out code.

For sake of readability.

Fix crash if connection is closed while sending a query to the server.

This is a similar bug that was fixed in commit 22b151eb. Some functions
reset conn->sock to NULL on a socket error, so we must not keep a copy of
it in a local variable. Also, SOCK_get_id should check for NULL argument
like all the rest of the SOCK_get/put_* functions.

Remove unused NDEBUG and PSQLODBC_EXPORTS defines from makefiles.

Remove more unused stuff from win64.mak

Silence misc issues flagged up by Clang's static analyzer, scan-build

Using tolower() requires ctype.h with some compilers.

gcc and MSVC were happy without it, but clang warns.

Merge 32-bit and 64-bit .wxs files used to build the .MSM files on Windows

Unify the 32-bit and 64-bit installer build processes.

Use the same .wxs file for both builds. The MSM files is still generated
differently, but this is a step in the right direction.

Fix Debug-mode builds with win64.mak

It apparently hasn't worked for a very long time. Copy some tricks from
the already-deleted win32.mak to make it work: Use /MDd rather than /MD
when compiling in Debug mode, and replace /GZ with /RTC1, to silence a
deprecation warning from the compiler

Fix Windows build with MSDTC=no.

Rename Windows output folders, for clarity.

Remove obsolete Windows build files.

buildx86.ps1 and buildx64.ps1 were used to invoke the nmake makefiles, but
have been obsoleted by the scripts in winbuild/. win64.mak can be used to
build 32-bit and 64-bit binaries now, so win32.mak is no longer needed.

Simplify win32.mak and win64.mak a little bit.

Change the way we test whether a directory exists, so that we don't need
a version check for Windows NT. (although we probably don't need to support
compiling on NT anymore anyway)

Improve Windows compilation instructions in the docs.

Refactor win64.mak to allow passing options in local file.

Move all the default values for external library paths to
windows-defaults.mak. Add an include for file called windows-local.mak,
this allows you to put local config options in that file, which is much
easier than passing them on the command line every time.

Remove /incremental option from linker call on Windows / nmake

The linker defaults to what were passing, so should work fine without it.

Also remove an unused variable from win32.mak

Remove special check for nmake version 6.00.9782.0

Surely we don't need to support that old versions anymore. Besides, it
was bizarre to check for that exact version, and not any older or newer
versions of that era.

Remove unused cruft from Windows build files.

Add comments to modify_msi.vbs.

It took me quite a while to reverse-engineer what it does, and more
importantly, why. Add comment to save time for the next reader.

Clean up Windows makefiles

Get rid of the CPU and CPUTYPE variables in win64.mak. Instead, assume that
the TARGET_CPU variable has been set correctly in the environment.
TARGET_CPU is set by "setenv" script that comes with the Microsoft Visual
Studio C compiler package. It never worked to pass a "CPU=x86" and using a
x64 compiler, for example, so seems better to determine the correct target
from the environment, than fail with a cryptic error message if the CPU
argument and the environment don't match. To build 32-bit binaries with
win64.mak, simply do "setenv /x86" before running "nmake -f win64.mak".
Similarly, to build 64-bit binaries, use "setenv /x64".

It should now be possible to build both 32-bit and 64-bit binaries with
win64.mak. Ideally, we could now remove win32.mak altogether, but there are
small differences between win64.mak and win32.mak in the build options that
they use. I don't understand the differences enough to dare to remove
win32.mak yet.

The CPU=AMD64 alias for x64 is no longer supported (as the whole parameter
is now gone).

Also do whitespace & other cosmetic fixes to win32.mak and win64, to make
them more similar. This helps when comparing the two with "diff".

Remove remains of the long-gone "MULTIBYTE" setting (it was misspelled in
win64.mak, anyway)

Rip out broken retry/timeout logic in SOCK_wait_for_ready.

At a quick glance, the logic looked like retry with progressive timeout, but
it was horribly broken. First of all, the timeout was only used if
retry_count was passed as 0. And in that case, the timeout was 0. So
retry_count == 0 actually meant "check if data can be read/written without
blocking", while retry_count > 0 meant "sleep until data is available". There
was also some code to handle retry_count < 0, but that was dead code because
none of the callers passed a negative value.

The hardcoded max. retry count of 30 retries was quite bogus too. Filling
the output buffer more 30 times is not necessary an error, and it's not
clear that the ssl library could not return SSL_ERROR_WANT_READ or
SSL_ERROR_WANT_WRITE more than 30 times. Unlikely, yes, but possible.

So, just rip out the retry_count and progressive timeout logic. Replace
the retry_count argument with a simple "should block?" argument. I didn't
dare to remove the condition that we always wait if SSL is enabled, although
I don't understand that at all. But perhaps there's some well-hidden reason
for that.

Per report from Prakash Itnal, although this just removes the broken code,
and won't enable the kind of a timeout that he wanted.

Fix broken expected output of catalogfunctions regression test.

Add Windows build directories to .gitignore

Michael Paquier

corresponds to notice the message level change.

Fixed test/expected.

Prep release ver 09.03.0300

Fix typo in error message.

Fix crash if connection is closed during CC_send_query_append function.

Malcolm MacLeod

Fix two UseDeclareFetch bugs.

1. NOTICE messages were not delivered to the application, if they arrived
as response to the DECLARE CURSOR statement.

2. Array-bound parameters on SELECT-queries caused a "cursor already open"
error.

These bugs were found by running the regression suite with
UseDeclareFetch=1. It's now clean.

Fix typo in test case

Don't clear error number when SQLGetDiagRec is called.

Calls to SQLGetDiagRec are supposed to be nondestructive, per ODBC spec.
Also add a test case for that.

Print a better error message in regression tests on an invalid handle.

Has no effect at the moment, but I've seen it happen during development
of new regression tests, when I've passed an invalid handle to print_diag
by accident.

Add test case for error-rollback behavior.

The rollback behavior is specified in the Protocol setting.

Michael Paquier

Avoid deprecated SQLAllocStmt/SQLFreeStmt in regression tests.

The deprecated functions have a test case of their own.

Michael Paquier

Add test case for SQLGetConnectOption

Michael Paquier

Add a new test case to test ODBC functions deprecated in ODBC 3.0

Michael Paquier

Add more test cases for ODBC catalog functions.

Michael Paquier

Add test/Makefile to .gitignore.

It's generated from test/Makefile.in now.

Use the PG_CONFIG setting from ./configure cmd line in regression tests.

If compiling without libpq (configure --without-libpq), then you still need
to have pg_config in path to build the regression tests, or specify
PG_CONFIG at the make command line.

Replace NOT_USE_LIBPQ with USE_LIBPQ #define.

This avoids the cumbersome double-negatives, "#ifndef NOT_USE_LIBPQ",
making the code easier to read.

Use the same CFLAGS/LDFLAGS to build regression tests as used to build driver.

Add missing function declarations.

These are not actually used outside of pgtypes.c, but let's be consistent
and declare them in pgtypes.h anyway.

Michael Paquier

Use "function(void)" instead of "function()" to declare 0-arg functions.

"function(void)" is the correct syntax in a function declaration, although
in practice compilers accept the latter too. It's OK in a function
definition, but change those too for consistency.

Michael Paquier and me.

Remove a couple of unused functions, and fix a typeo.

More whitespace cleanup.

Make whitespace and indentation more consistent in SQL_API function
declarations.

Whitespace and indentation fixes.

This also changed the file permissions stored in git for a couple of header
files, removing the executable bit. Didn't realize they were different from
all the others, but apparently my editor decided to change them. Now
they're the same as all the other files, which seems good.

Michael Paquier and me.

Pass the argument to isalpha/isspace etc. routines as unsigned char.

Per the C standard, the routine should be passed an int, with a value that's
representable as an unsigned char or EOF. Passing a signed char is wrong,
because a negative value is not representable as an unsigned char.
Unfortunately no compiler warns about that.

Fix implicit casts between SQLCHAR and char. Don't rely on -Wno-pointer-sign.

Prep release 09.03.0210.

Add a .gitignore file to ignore test/results dir.

Michael Paquier

Use "exit(1);" instead of "return;" to exit from main().

Per Michael Paquier, at least some compiler on OS X didn't like it.

Remove some dead code.

Pass the content of POSTGRES_RESOURCE_VERSION variable to the resource
compiler correctly so that FileVersion and ProductVersion are properly set.

Silence another compiler warning.

conn_settings cannot be NULL here.

Silence compiler warning.

Michael Paquier

Record the result of Build in configuration.xml.

It's preferable to leave PRODUCTCODE history.

Add test case for CTE queries (WITH ...)

Tests for the bug fixed by commit b547553233b52fa5a84c24801c468530d8dd9a18.
None of the existing test cases covered that, so seems good to add one. This
test case gives a different error message than the original one, but it's
the same underlying issue.

Fix encoding of release-7.3.html

The Content-type meta tag claimed it to be US-ASCII, but there was in fact
one non-ASCII character in there. Change the claimed encoding to UTF-8, and
fix the special character to the right byte sequence in UTF-8.

Convert line-endings from CT+LF to LF.

All the other files were using LF, so let's be consistent.

Fix locking in SC_set_prepared

Commit 2f4f8eabf6ce685c38d69da3e98132b77d38d188 added ENTER/LEAVE_CRIT_CS
calls in SC_set_prepared, as a belt-and-suspenders fix for the race condition
already fixed by commit 7065a7af196bb09f80beddce82fa728c8a6dfcba. But it
didn't take a NULL connection into account.

Update EXTRA_DIST in Makefile.am

Files for the new lfconversion test case were missing. Also, move bindcol
in the right place to keep the list in alphabetical order.

SSL verify[-(ca|full)] is avaiable since 8.4. There seems no need to
check it. Also there's no need to call lt_dlopen currently.

Currently PQConnectdbParams() not yet used anyway.

change version number.

change version number.

change version number.

Ooops missing change.

Forget win_md5.c.

Prep release ver 09.03.0200.

Prep release ver 09.03.0200.

Additional modules, and etc.

winresrc.h instead of winres.h.

Change to supply non-NULL parameters for SC_set_error().

It's safe to lock SC_set_prepared() itself.

The next release allows minor version up from 09.03.0100.

Fix compilation errors caused by the commit 3116f70ab51d0218ceff241fd1323f436bb4fd60.

The 2dn argument of SC_set_prepared() is int(enum) not BOOL.

Fix the judgement condition of minor or major upgrade.

Add locking to SQLFreeStmt and SQLFreeHandle.

This fixes a race condition, where SQLFreeStmt is called while the
connection is busy executing another statement. SC_set_prepared would see
that the connection is busy (CONN_EXECUTING), and not issue a DEALLOCATE
statement to free the prepared statement in the backend, leaking it.

Refactor utf8_to_ucs_lf.

A macro is difficult to debug, so turn it into a regular function.

Also, add a new test case for LF->CR+LF conversion, to test the bug that
Nils Gösche reported (which was already fixed).

Revert "When LF->CR+LF conversion causes an buffer truncation, supress the conversion (in case of unicode)."
This reverts commit 3666c87c1440862bde2e6b8f43ee585deed70d31.

The driver takes SQL_C_LONG to mean SQLINTEGER rather than "long".

The regression test was failing on the s390x architecture because of that.
It's big-endian, with sizeof(long) == 8.

Add regression test for SQLBindCol.

We were already doing SQLBindCol as part of the positioned-update test, but
seems good to have one explicitly for it.

Fix cursors test case on big-endian systems.

The SQL_CURSOR_COMMIT_BEHAVIOR property is a SQLUSMALLINT, not SQLUINTEGER.
On a little-endian system, you wouldn't notice, provided that the target
variable was initialized to 0 before the SQLGetInfo call.

Per report from Christoph Berg that the cursors test was failing on mipsel
and other big-endian architectures.

MSBuild instead of nmake to build binaries using 10.0 or later version of VC.
See readme_winbuild.txt or winbuild/readme.txt about the use.

Set TCP keepalive by default.