`mod lf_apply`: clean up code and use slices instead of raw pointers - #728

Merged

fbossen merged 26 commits into

memorysafety:mainfrom

fbossen:cleanup-lf-apply

Mar 7, 2024

Commits on Mar 6, 2024

`fn rav1d_copy_lpf`: use slice instead of pointer for `src_stride`
Frank Bossen
committed
64c507e
`fn rav1d_copy_lpf`: use slice instead of pointer for `lr_stride`
Frank Bossen
committed
5c37e6c
`fn rav1d_copy_lpf`: make `src` parameter a slice reference
Frank Bossen
committed
d769725
`fn rav1d_loopfilter_sbrow_cols`: use slice reference instead of raw pointer
Frank Bossen
committed
602b043
`fn rav1d_loopfilter_sbrow_cols`: clean up loops
Frank Bossen
committed
87342bd
`fn rav1d_loopfilter_sbrow_cols`: avoid repeated casts to `usize`
Frank Bossen
committed
7935e6e
`fn rav1d_loopfilter_sbrow_cols`: clean up masks
Frank Bossen
committed
6e15020
Use slice instead of raw pointer
Frank Bossen
committed
3063ae1
`fn rav1d_loopfilter_sbrow_cols`: make `lpf_y` and `lpf_uv` slices
Frank Bossen
committed
61f83f1
Clean up types and casts
Frank Bossen
committed
f8daee4
Use references to `Rav1dDSPContext` object
Frank Bossen
committed
689fa49
`fn rav1d_loopfilter_sbrow_cols`: pass slices of samples as argument
Frank Bossen
committed
b0c447a
`fn filter_plane_cols_y/uv`: make `dst`/`u`/`v` arguments a slice
Frank Bossen
committed
7666d9e
`lf_apply.rs`: use slices instead of raw pointers
Frank Bossen
committed
78ee731
`fn backup_lpf`; use slice as parameter
Frank Bossen
committed
f0fd40b
Define inner blocks for computation of `p` and `p_offset`
Frank Bossen
committed
68f0817
`lflvl`: use slice instead of raw pointer
Frank Bossen
committed
789c78f
`fn backup_lpf`: use slice instead of raw pointer for `dst`
Frank Bossen
committed
ad49d8d
`fn filter_plane_rows_y/uv`: use slice instead of raw pointer
Frank Bossen
committed
e4d2966
Address comments from review
Frank Bossen
committed
f54cb43
Copy comments from C code
Frank Bossen
committed
060574d
Avoid unnecessary type casting when invoking `BD::pxstride`
Frank Bossen
committed
793a00f
Reduce number of casts for variable `top`
Frank Bossen
committed
47fb8fa
`fn backup_lpf`: clean up pixel copy
Frank Bossen
committed
d639377
Remove explicit dereference of `dsp`
Frank Bossen
committed
c509c30
Clean up use of `split_at_mut` and pixel copy
Frank Bossen
committed
5a24dbd

Navigation Menu

`mod lf_apply`: clean up code and use slices instead of raw pointers - #728

`mod lf_apply`: clean up code and use slices instead of raw pointers#728
fbossen merged 26 commits into
memorysafety:mainfrom
fbossen:cleanup-lf-apply

Commits on Mar 6, 2024

`fn rav1d_copy_lpf`: use slice instead of pointer for `src_stride`

`fn rav1d_copy_lpf`: use slice instead of pointer for `lr_stride`

`fn rav1d_copy_lpf`: make `src` parameter a slice reference

`fn rav1d_loopfilter_sbrow_cols`: use slice reference instead of raw pointer

`fn rav1d_loopfilter_sbrow_cols`: clean up loops

`fn rav1d_loopfilter_sbrow_cols`: avoid repeated casts to `usize`

`fn rav1d_loopfilter_sbrow_cols`: clean up masks

Use slice instead of raw pointer

`fn rav1d_loopfilter_sbrow_cols`: make `lpf_y` and `lpf_uv` slices

Clean up types and casts

Use references to `Rav1dDSPContext` object

`fn rav1d_loopfilter_sbrow_cols`: pass slices of samples as argument

`fn filter_plane_cols_y/uv`: make `dst`/`u`/`v` arguments a slice

`lf_apply.rs`: use slices instead of raw pointers

`fn backup_lpf`; use slice as parameter

Define inner blocks for computation of `p` and `p_offset`

`lflvl`: use slice instead of raw pointer

`fn backup_lpf`: use slice instead of raw pointer for `dst`

`fn filter_plane_rows_y/uv`: use slice instead of raw pointer

Address comments from review

Copy comments from C code

Avoid unnecessary type casting when invoking `BD::pxstride`

Reduce number of casts for variable `top`

`fn backup_lpf`: clean up pixel copy

Remove explicit dereference of `dsp`

Clean up use of `split_at_mut` and pixel copy