Re: oledb32: Add IDataSourceLocator support (try 6)

2013-08-21 Thread Nikolay Sivov
On 8/21/2013 10:33, Alistair Leslie-Hughes wrote: +typedef struct DataSourceImpl +{ +IDataSourceLocator IDataSourceLocator_iface; +LONG ref; + +} DataSourceImpl; All good, except this one.

Re: oledb32: Add IDataSourceLocator support (try 5)

2013-08-20 Thread Nikolay Sivov
x27;t need these: +#include "wine/unicode.h" +#include "wine/list.h" Changelog: oledb32: Add IDataSourceLocator support Best Regards Alistair Leslie-Hughes

Re: oledb32: Add IDataSourceLocator support (try 4)

2013-08-20 Thread Nikolay Sivov
On 8/21/2013 09:15, Alistair Leslie-Hughes wrote: Hi, Use helper functions for memory Changelog: oledb32: Add IDataSourceLocator support Best Regards Alistair Leslie-Hughes It looks okay, but I suggest to fix a naming: +typedef struct DataSourceImpl +{ +IDataSourceLocator

Re: oledb32: Add IDataSourceLocator support

2013-08-01 Thread Alistair Leslie-Hughes
Ignore this patch On 1/08/2013 6:36 PM, Alistair Leslie-Hughes wrote: Hi, Changelog: oledb32: Add IDataSourceLocator support Best Regards Alistair Leslie-Hughes