Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix lint failure #59

Merged
merged 1 commit into from
Sep 11, 2023
Merged

Fix lint failure #59

merged 1 commit into from
Sep 11, 2023

Conversation

kou
Copy link
Member

@kou kou commented Sep 11, 2023

fix #58

error: casting raw pointers to the same type and constness is unnecessary (`*mut capi::DFArrowArray` -> `*mut capi::DFArrowArray`)
   --> src/capi.rs:651:21
    |
651 |                     (*c_abi_record_batch as *mut DFArrowArray) as *mut FFI_ArrowArray,
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `*c_abi_record_batch`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast
    = note: `-D clippy::unnecessary-cast` implied by `-D clippy::all`
@kou
Fix lint failure
10ea52c 
fix datafusion-contrib#58

    error: casting raw pointers to the same type and constness is unnecessary (`*mut capi::DFArrowArray` -> `*mut capi::DFArrowArray`)
       --> src/capi.rs:651:21
        |
    651 |                     (*c_abi_record_batch as *mut DFArrowArray) as *mut FFI_ArrowArray,
        |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `*c_abi_record_batch`
        |
        = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast
        = note: `-D clippy::unnecessary-cast` implied by `-D clippy::all`
@kou kou merged commit 99c2d7f into datafusion-contrib:main Sep 11, 2023
11 of 13 checks passed
@kou kou deleted the fix-lint branch September 11, 2023 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant
@kou