[ 
https://issues.apache.org/jira/browse/ARROW-2415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16429902#comment-16429902
 ] 

ASF GitHub Bot commented on ARROW-2415:
---------------------------------------

maxim-lian commented on issue #1851: ARROW-2415: [Rust] Fix clippy 
ref-match-pats warnings.
URL: https://github.com/apache/arrow/pull/1851#issuecomment-379588224
 
 
   > it should be self.bits.is_empty() hopefully rather than self.len != 0
   
   👍 
   
   > I'd planned to do subsequent things separately 
   
   That's fine IMO. For the immediate fixes I don't think there's much 
ambiguity / need for discussion, but feel free to split up regardless. 
   
   Should we add a check in CI? 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [Rust] Fix using references in pattern matching
> -----------------------------------------------
>
>                 Key: ARROW-2415
>                 URL: https://issues.apache.org/jira/browse/ARROW-2415
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Rust
>            Reporter: Bruce Mitchener
>            Priority: Major
>              Labels: pull-request-available
>
> Clippy reports 
> [https://rust-lang-nursery.github.io/rust-clippy/v0.0.191/index.html#match_ref_pats]
>  warnings.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to