[PR #1730] [MERGED] [Rust]Ported AVC Module to Rust #2451

Closed
opened 2026-01-29 17:22:13 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/1730
Author: @steel-bucket
Created: 8/17/2025
Status: Merged
Merged: 9/6/2025
Merged by: @prateekmedia

Base: masterHead: migration-avc-module


📝 Commits (8)

  • af262db AVC Module: ported AVC Module to Rust
  • c2d36a9 AVC module: Minor semantic changes
  • bab3580 AVC Module: Failing CI
  • c721516 Merge branch 'CCExtractor:master' into migration-avc-module
  • 1e2c2e2 AVC Module: SIMD Optimisations
  • f3f4cca AVC Module: Optimization in SEI
  • 2186738 AVC Module: removed panic
  • 443043a Merge branch 'CCExtractor:master' into migration-avc-module

📊 Changes

11 files changed (+2315 additions, -378 deletions)

View changed files

📝 src/lib_ccx/avc_functions.c (+7 -0)
📝 src/rust/build.rs (+4 -0)
📝 src/rust/lib_ccxr/src/common/constants.rs (+1 -0)
src/rust/src/avc/common_types.rs (+311 -0)
src/rust/src/avc/core.rs (+506 -0)
src/rust/src/avc/mod.rs (+82 -0)
src/rust/src/avc/nal.rs (+641 -0)
src/rust/src/avc/sei.rs (+337 -0)
📝 src/rust/src/decoder/mod.rs (+5 -2)
📝 src/rust/src/decoder/service_decoder.rs (+26 -7)
📝 src/rust/src/lib.rs (+395 -369)

📄 Description

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.
  • I have mentioned this change in the changelog.

My familiarity with the project is as follows (check one):

  • I have never used CCExtractor.
  • I have used CCExtractor just a couple of times.
  • I absolutely love CCExtractor, but have not contributed previously.
  • I am an active contributor to CCExtractor.

This PR fixes the AVC functionality which was broken and migrates the entire avc_functions.c and avc_functions.h to Rust.

tested on This AVC stream


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/CCExtractor/ccextractor/pull/1730 **Author:** [@steel-bucket](https://github.com/steel-bucket) **Created:** 8/17/2025 **Status:** ✅ Merged **Merged:** 9/6/2025 **Merged by:** [@prateekmedia](https://github.com/prateekmedia) **Base:** `master` ← **Head:** `migration-avc-module` --- ### 📝 Commits (8) - [`af262db`](https://github.com/CCExtractor/ccextractor/commit/af262db01163d6c6bdac3dbfab961052648965c1) AVC Module: ported AVC Module to Rust - [`c2d36a9`](https://github.com/CCExtractor/ccextractor/commit/c2d36a9e1aec187b70d90422f70d265fd02b3489) AVC module: Minor semantic changes - [`bab3580`](https://github.com/CCExtractor/ccextractor/commit/bab35808277fc6b450d6e94971bae268ab9494e3) AVC Module: Failing CI - [`c721516`](https://github.com/CCExtractor/ccextractor/commit/c7215169689271f458f9c7e517a469a619a671c8) Merge branch 'CCExtractor:master' into migration-avc-module - [`1e2c2e2`](https://github.com/CCExtractor/ccextractor/commit/1e2c2e25548bd09eceeed3d3eb6d8d7d3b51890d) AVC Module: SIMD Optimisations - [`f3f4cca`](https://github.com/CCExtractor/ccextractor/commit/f3f4cca3dec06e8847569ff74eb5a21aa7b31362) AVC Module: Optimization in SEI - [`2186738`](https://github.com/CCExtractor/ccextractor/commit/2186738313d592c9ad801ce7d5acde75f3ef1303) AVC Module: removed panic - [`443043a`](https://github.com/CCExtractor/ccextractor/commit/443043ad6a2998ae203fb01d65807948a2974532) Merge branch 'CCExtractor:master' into migration-avc-module ### 📊 Changes **11 files changed** (+2315 additions, -378 deletions) <details> <summary>View changed files</summary> 📝 `src/lib_ccx/avc_functions.c` (+7 -0) 📝 `src/rust/build.rs` (+4 -0) 📝 `src/rust/lib_ccxr/src/common/constants.rs` (+1 -0) ➕ `src/rust/src/avc/common_types.rs` (+311 -0) ➕ `src/rust/src/avc/core.rs` (+506 -0) ➕ `src/rust/src/avc/mod.rs` (+82 -0) ➕ `src/rust/src/avc/nal.rs` (+641 -0) ➕ `src/rust/src/avc/sei.rs` (+337 -0) 📝 `src/rust/src/decoder/mod.rs` (+5 -2) 📝 `src/rust/src/decoder/service_decoder.rs` (+26 -7) 📝 `src/rust/src/lib.rs` (+395 -369) </details> ### 📄 Description <!-- Please prefix your pull request with one of the following: **[FEATURE]** **[FIX]** **[IMPROVEMENT]**. --> **In raising this pull request, I confirm the following (please check boxes):** - [x] I have read and understood the [contributors guide](https://github.com/CCExtractor/ccextractor/blob/master/.github/CONTRIBUTING.md). - [x] I have checked that another pull request for this purpose does not exist. - [x] I have considered, and confirmed that this submission will be valuable to others. - [x] I accept that this submission may not be used, and the pull request closed at the will of the maintainer. - [x] I give this submission freely, and claim no ownership to its content. - [ ] **I have mentioned this change in the [changelog](https://github.com/CCExtractor/ccextractor/blob/master/docs/CHANGES.TXT).** **My familiarity with the project is as follows (check one):** - [ ] I have never used CCExtractor. - [ ] I have used CCExtractor just a couple of times. - [ ] I absolutely love CCExtractor, but have not contributed previously. - [x] I am an active contributor to CCExtractor. --- This PR fixes the AVC functionality which was broken and migrates the entire `avc_functions.c` and `avc_functions.h` to Rust. tested on [This AVC stream](https://drive.google.com/file/d/1GsxXE3EW9r9UfsRkoLlP2k03p6gLGbLe/view?usp=sharing) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 17:22:13 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#2451