eitsupi commented on code in PR #3099: URL: https://github.com/apache/arrow-adbc/pull/3099#discussion_r2199534491
########## rust/core/src/driver_manager.rs: ########## @@ -1310,3 +1645,506 @@ impl Drop for ManagedStatement { unsafe { method(statement.deref_mut(), null_mut()) }; } } + +#[cfg(target_os = "windows")] +extern crate windows_sys as windows; Review Comment: I'm not an expert on this, but at least here I think there's no need to use `extern crate` and just `use` is sufficient. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org