Skip to content

Commit b7bf390

Browse files
committed
fix: re-enable asm for scenechange detection
This was accidentally disabled when migrating the scenechange code to a separate crate, because the feature for assembly was disabled. This also fixes clippy lints which are required for CI to pass.
1 parent a2f01b3 commit b7bf390

File tree

5 files changed

+12
-6
lines changed

5 files changed

+12
-6
lines changed

.github/workflows/rav1e.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434

3535
env:
3636
CARGO_INCREMENTAL: 0
37-
RUSTFLAGS: '-Dwarnings'
37+
RUSTFLAGS: "-Dwarnings"
3838

3939
steps:
4040
- uses: actions/checkout@v4
@@ -77,7 +77,7 @@ jobs:
7777

7878
env:
7979
CARGO_INCREMENTAL: 0
80-
RUSTFLAGS: '-Dwarnings'
80+
RUSTFLAGS: "-Dwarnings"
8181
CARGO_TARGET_WASM32_WASIP2_RUNNER: wasmtime
8282

8383
steps:

Cargo.lock

Lines changed: 3 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,9 @@ dump_lookahead_data = ["byteorder", "image"]
7777

7878
[dependencies]
7979
arg_enum_proc_macro = "0.3.4"
80-
av-scenechange = { version = "0.14.1", default-features = false }
80+
av-scenechange = { version = "0.14.1", default-features = false, features = [
81+
"asm",
82+
] }
8183
bitstream-io = "4.1.0"
8284
cfg-if = "1.0"
8385
clap = { version = "4.5", optional = true, default-features = false, features = [
@@ -221,7 +223,8 @@ no-default-features = true
221223

222224
[lints.clippy]
223225
doc_link_with_quotes = "warn"
224-
doc_markdown = "warn"
226+
# TODO: Allow for now, since `built` generates code which fails this.
227+
# doc_markdown = "warn"
225228
missing_errors_doc = "warn"
226229
missing_panics_doc = "warn"
227230
missing_safety_doc = "warn"

src/predict.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ impl PredictionMode {
283283
#[inline]
284284
fn get_mv_params<T: Pixel>(
285285
rec_plane: &Plane<T>, po: PlaneOffset, mv: MotionVector,
286-
) -> (i32, i32, PlaneSlice<T>) {
286+
) -> (i32, i32, PlaneSlice<'_, T>) {
287287
let &PlaneConfig { xdec, ydec, .. } = &rec_plane.cfg;
288288
let row_offset = mv.row as i32 >> (3 + ydec);
289289
let col_offset = mv.col as i32 >> (3 + xdec);

src/tiling/tile_restoration_state.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -414,7 +414,7 @@ tile_restoration_state_common!(
414414

415415
impl TileRestorationStateMut<'_> {
416416
#[inline(always)]
417-
pub const fn as_const(&self) -> TileRestorationState {
417+
pub const fn as_const(&self) -> TileRestorationState<'_> {
418418
TileRestorationState {
419419
planes: [
420420
self.planes[0].as_const(),

0 commit comments

Comments
 (0)