Skip to content

Commit ffd5a0d

Browse files
committed
slicelen -> slice_len
1 parent 119913b commit ffd5a0d

File tree

3 files changed

+24
-24
lines changed

3 files changed

+24
-24
lines changed

stdlib/src/mmap.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -934,11 +934,11 @@ mod mmap {
934934
slice: &SaturatedSlice,
935935
vm: &VirtualMachine,
936936
) -> PyResult<PyObjectRef> {
937-
let (range, step, slicelen) = slice.adjust_indices(self.len());
937+
let (range, step, slice_len) = slice.adjust_indices(self.len());
938938

939939
let mmap = self.check_valid(vm)?;
940940

941-
if slicelen == 0 {
941+
if slice_len == 0 {
942942
return Ok(PyBytes::from(vec![]).into_ref(vm).into());
943943
} else if step == 1 {
944944
let bytes = match mmap.deref().as_ref().unwrap() {
@@ -948,7 +948,7 @@ mod mmap {
948948
return Ok(PyBytes::from(bytes.to_vec()).into_ref(vm).into());
949949
}
950950

951-
let mut result_buf = Vec::with_capacity(slicelen);
951+
let mut result_buf = Vec::with_capacity(slice_len);
952952
if step.is_negative() {
953953
for i in range.rev().step_by(step.unsigned_abs()) {
954954
let b = match mmap.deref().as_ref().unwrap() {
@@ -1017,15 +1017,15 @@ mod mmap {
10171017
value: PyObjectRef,
10181018
vm: &VirtualMachine,
10191019
) -> PyResult<()> {
1020-
let (range, step, slicelen) = slice.adjust_indices(zelf.len());
1020+
let (range, step, slice_len) = slice.adjust_indices(zelf.len());
10211021

10221022
let bytes = bytes_from_object(vm, &value)?;
10231023

1024-
if bytes.len() != slicelen {
1024+
if bytes.len() != slice_len {
10251025
return Err(vm.new_index_error("mmap slice assignment is wrong size".to_owned()));
10261026
}
10271027

1028-
if slicelen == 0 {
1028+
if slice_len == 0 {
10291029
// do nothing
10301030
Ok(())
10311031
} else if step == 1 {

vm/src/builtins/memory.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -479,7 +479,7 @@ impl PyMemoryView {
479479
fn init_slice(&mut self, slice: &PySlice, dim: usize, vm: &VirtualMachine) -> PyResult<()> {
480480
let (shape, stride, _) = self.desc.dim_desc[dim];
481481
let slice = slice.to_saturated(vm)?;
482-
let (range, step, slicelen) = slice.adjust_indices(shape);
482+
let (range, step, slice_len) = slice.adjust_indices(shape);
483483

484484
let mut is_adjusted_suboffset = false;
485485
for (_, _, suboffset) in self.desc.dim_desc.iter_mut().rev() {
@@ -498,7 +498,7 @@ impl PyMemoryView {
498498
range.start
499499
};
500500
}
501-
self.desc.dim_desc[dim].0 = slicelen;
501+
self.desc.dim_desc[dim].0 = slice_len;
502502
self.desc.dim_desc[dim].1 *= step;
503503

504504
Ok(())

vm/src/sliceable.rs

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -44,16 +44,16 @@ where
4444
slice: SaturatedSlice,
4545
items: &[Self::Item],
4646
) -> PyResult<()> {
47-
let (range, step, slicelen) = slice.adjust_indices(self.as_ref().len());
48-
if slicelen != items.len() {
47+
let (range, step, slice_len) = slice.adjust_indices(self.as_ref().len());
48+
if slice_len != items.len() {
4949
Err(vm
5050
.new_buffer_error("Existing exports of data: object cannot be re-sized".to_owned()))
5151
} else if step == 1 {
5252
self.do_set_range(range, items);
5353
Ok(())
5454
} else {
5555
self.do_set_indexes(
56-
SaturatedSliceIter::from_adjust_indices(range, step, slicelen),
56+
SaturatedSliceIter::from_adjust_indices(range, step, slice_len),
5757
items,
5858
);
5959
Ok(())
@@ -66,21 +66,21 @@ where
6666
slice: SaturatedSlice,
6767
items: &[Self::Item],
6868
) -> PyResult<()> {
69-
let (range, step, slicelen) = slice.adjust_indices(self.as_ref().len());
69+
let (range, step, slice_len) = slice.adjust_indices(self.as_ref().len());
7070
if step == 1 {
7171
self.do_set_range(range, items);
7272
Ok(())
73-
} else if slicelen == items.len() {
73+
} else if slice_len == items.len() {
7474
self.do_set_indexes(
75-
SaturatedSliceIter::from_adjust_indices(range, step, slicelen),
75+
SaturatedSliceIter::from_adjust_indices(range, step, slice_len),
7676
items,
7777
);
7878
Ok(())
7979
} else {
8080
Err(vm.new_value_error(format!(
8181
"attempt to assign sequence of size {} to extended slice of size {}",
8282
items.len(),
83-
slicelen
83+
slice_len
8484
)))
8585
}
8686
}
@@ -95,16 +95,16 @@ where
9595
}
9696

9797
fn del_item_by_slice(&mut self, _vm: &VirtualMachine, slice: SaturatedSlice) -> PyResult<()> {
98-
let (range, step, slicelen) = slice.adjust_indices(self.as_ref().len());
99-
if slicelen == 0 {
98+
let (range, step, slice_len) = slice.adjust_indices(self.as_ref().len());
99+
if slice_len == 0 {
100100
Ok(())
101101
} else if step == 1 {
102102
self.do_set_range(range, &[]);
103103
Ok(())
104104
} else {
105105
self.do_delete_indexes(
106106
range.clone(),
107-
SaturatedSliceIter::from_adjust_indices(range, step, slicelen).positive_order(),
107+
SaturatedSliceIter::from_adjust_indices(range, step, slice_len).positive_order(),
108108
);
109109
Ok(())
110110
}
@@ -187,8 +187,8 @@ pub trait SliceableSequenceOp {
187187
_vm: &VirtualMachine,
188188
slice: SaturatedSlice,
189189
) -> PyResult<Self::Sliced> {
190-
let (range, step, slicelen) = slice.adjust_indices(self.len());
191-
let sliced = if slicelen == 0 {
190+
let (range, step, slice_len) = slice.adjust_indices(self.len());
191+
let sliced = if slice_len == 0 {
192192
Self::empty()
193193
} else if step == 1 {
194194
if step.is_negative() {
@@ -395,13 +395,13 @@ impl SaturatedSlice {
395395
saturate_index(self.start, len)..saturate_index(self.stop, len)
396396
};
397397

398-
let (range, slicelen) = if range.start >= range.end {
398+
let (range, slice_len) = if range.start >= range.end {
399399
(range.start..range.start, 0)
400400
} else {
401-
let slicelen = (range.end - range.start - 1) / self.step.unsigned_abs() + 1;
402-
(range, slicelen)
401+
let slice_len = (range.end - range.start - 1) / self.step.unsigned_abs() + 1;
402+
(range, slice_len)
403403
};
404-
(range, self.step, slicelen)
404+
(range, self.step, slice_len)
405405
}
406406

407407
pub fn iter(&self, len: usize) -> SaturatedSliceIter {

0 commit comments

Comments
 (0)