Re: Strings again :(

Reading the prototypes of SQLGetInstalledDrivers in the header file
<odbcinst.h> (in VS2008):

BOOL INSTAPI SQLGetInstalledDrivers
(
    __out_ecount(cchBufMax) LPSTR lpszBuf,
    WORD cchBufMax,
    WORD* pcchBufOut
);

BOOL INSTAPI SQLGetInstalledDriversW
(
    __out_ecount(cchBufMax) LPWSTR lpszBuf,
    WORD cchBufMax,
    WORD* pcchBufOut
);

It is clear that the function SQLGetInstalledDrivers() wants the size of
buffer in *TCHAR's*; in fact, the Hungarian prefix 'cch' (instead of
'cb'), is used and tells us that the TCHAR count is expected.

So, Mihai was correct in suggesting the use of _countof() instead of
sizeof(), to make the code Unicode aware.

The real problem here is the wrong MSDN documentation

http://msdn.microsoft.com/en-us/library/ms714005.aspx

that uses this misleading ANSI-only prototype:

BOOL SQLGetInstalledDrivers(
     LPSTR lpszBuf,
     WORD cbBufMax,
     WORD * pcbBufOut);

I think that Tom was mislead by this documentation (if I read 'cb' prefix,
I think of sizeof(), too, like Tom did; instead, if I read 'cch' prefix, I
think about _countof()).

Giovanni