search

Giorgi / DuckDB.NET

Bindings and ADO.NET Provider for DuckDB
https://duckdb.net
MIT License
356 stars 62 forks source link

Replace deprecated fields with calls to bindings. #33

Closed kmosegaard closed 2 years ago

kmosegaard commented 2 years ago

This PR replace use of deprecated fields in DuckDBResult to the newly added recommended bindings (see https://github.com/duckdb/duckdb/blob/master/src/include/duckdb.h#L214).

Feedback is welcome.

Giorgi commented 2 years ago

@kmosegaard It looks like NativeMethods.DuckDBNullmaskData throws exception when the column > 0. Any ideas?

kmosegaard commented 2 years ago

I found the problem. DuckDBResult was not decorated with [In, Out]. I've created another PR.