LPBYTE typedef in sqlfront.h and php_mssql.c

Stuart Henderson <[email protected]>
Newsgroups gmane.comp.db.tds.freetds
Message-ID <[email protected]>
http://freetds.cvs.sourceforge.net/viewvc/freetds/freetds/include/sqlfront.h?r1=1.5&r2=1.6&view=patch

include/sqlfront.h r1.6 added a number of typedefs for "Better Win32
compatibility", including this:

+typedef       BYTE *	LPBYTE;

It breaks build of php_mssql, would it be appropriate to #ifdef this?

In file included from /usr/obj/ports/php-5.2.17/php-5.2.17/ext/mssql/php_mssql.c:33:
/usr/obj/ports/php-5.2.17/php-5.2.17/ext/mssql/php_mssql.h:73: error: redefinition of typedef 'LPBYTE'
/usr/local/include/sqlfront.h:34: error: previous declaration of 'LPBYTE' was here
/usr/obj/ports/php-5.2.17/php-5.2.17/ext/mssql/php_mssql.c: In function 'php_mssql_do_connect':
/usr/obj/ports/php-5.2.17/php-5.2.17/ext/mssql/php_mssql.c:667: warning: cast from pointer to integer of different size
/usr/obj/ports/php-5.2.17/php-5.2.17/ext/mssql/php_mssql.c: In function 'php_mssql_get_column_content_without_type':
/usr/obj/ports/php-5.2.17/php-5.2.17/ext/mssql/php_mssql.c:1033: warning: passing argument 1 of 'spprintf' from incompatible pointer type

cheers,
Stuart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.