Skip to content

Use const fn where possible #5894

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

Merged
youknowone merged 17 commits into from
Jul 4, 2025
Merged

Use const fn where possible #5894

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
b30c919
vm
ShaharNaveh Jul 3, 2025
a18f0c2
compiler
ShaharNaveh Jul 3, 2025
fde322c
sre engine
ShaharNaveh Jul 3, 2025
80fba2f
compiler
ShaharNaveh Jul 3, 2025
5f790b8
common
ShaharNaveh Jul 3, 2025
8aef1a3
stdlib
ShaharNaveh Jul 3, 2025
5e08620
vm
ShaharNaveh Jul 3, 2025
4e91f6a
cargo fmt
ShaharNaveh Jul 3, 2025
593b9d2
Merge remote-tracking branch 'upstream/main' into constify
ShaharNaveh Jul 4, 2025
29b26d3
vm
ShaharNaveh Jul 4, 2025
6cd01bf
vm
ShaharNaveh Jul 4, 2025
6b1deb4
common
ShaharNaveh Jul 4, 2025
8adcaa5
compiler
ShaharNaveh Jul 4, 2025
dbf0078
compiler codegen
ShaharNaveh Jul 4, 2025
4b12536
vm
ShaharNaveh Jul 4, 2025
e55f32b
cargo fmt
ShaharNaveh Jul 4, 2025
eb3e2b3
remove bad consts
ShaharNaveh Jul 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
cargo fmt
  • Loading branch information
@ShaharNaveh
ShaharNaveh committed Jul 3, 2025
commit 4e91f6a036edefd285a54ffb8caf0d84236b8954
6 changes: 5 additions & 1 deletion common/src/encodings.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,11 @@ struct DecodeError<'a> {

/// # Safety
/// `v[..valid_up_to]` must be valid utf8
const unsafe fn make_decode_err(v: &[u8], valid_up_to: usize, err_len: Option<usize>) -> DecodeError<'_> {
const unsafe fn make_decode_err(
v: &[u8],
valid_up_to: usize,
err_len: Option<usize>,
) -> DecodeError<'_> {
let (valid_prefix, rest) = unsafe { v.split_at_unchecked(valid_up_to) };
let valid_prefix = unsafe { core::str::from_utf8_unchecked(valid_prefix) };
DecodeError {
Expand Down
7 changes: 6 additions & 1 deletion vm/src/builtins/traceback.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,12 @@ impl PyPayload for PyTraceback {

#[pyclass]
impl PyTraceback {
pub const fn new(next: Option<PyRef<Self>>, frame: FrameRef, lasti: u32, lineno: LineNumber) -> Self {
pub const fn new(
next: Option<PyRef<Self>>,
frame: FrameRef,
lasti: u32,
lineno: LineNumber,
) -> Self {
PyTraceback {
next: PyMutex::new(next),
frame,
Expand Down
Loading