Skip to content

Conversation

@ritchie46
Copy link
Member

No description provided.

@github-actions github-actions bot added internal An internal refactor or improvement python Related to Python Polars rust Related to Rust Polars labels Apr 19, 2025
#[cfg(feature = "nightly")]
pub fn select_unpredictable<T>(cond: bool, true_val: T, false_val: T) -> T {
cond.select_unpredictable(true_val, false_val)
core::intrinsics::select_unpredictable(cond, true_val, false_val)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rustc complained that the method didn't exist anymore, so we call the intrinsic directly.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this rust-lang/rust#139726.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, yeap. :)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be core::hint, not core::intrinsics.


pub fn is_sliced(&self) -> bool {
self.views.as_ptr() != self.views.storage_ptr()
!std::ptr::eq(self.views.as_ptr(), self.views.storage_ptr())
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New lint

@github-actions
Copy link
Contributor

The uncompressed lib size after this PR is 41.6759 MB.

@ritchie46 ritchie46 merged commit 64ee062 into main Apr 19, 2025
21 of 22 checks passed
@ritchie46 ritchie46 deleted the dev branch April 19, 2025 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal An internal refactor or improvement python Related to Python Polars rust Related to Rust Polars

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants