[PR #744] [CLOSED] Refactoring the source code and extension to python module. #1565

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

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/744
Author: @Diptanshu8
Created: 6/27/2017
Status: Closed

Base: masterHead: first_phase_evaluation


📝 Commits (10+)

  • d3540cc Made temp changes to initiate library coding
  • b0a0c92 library development done till api_start. Implementation of stop and status function is left.
  • 644d265 Facing VERSION attribute error in generated module
  • f70f34e pushing the api code
  • 9fa2e3e adding the ccextractorapi.py file
  • dba1d7b Debugging the core dumped error
  • 751a22f Resolving merge conflicts in .gitignore
  • 98e295e Commiting the build_library file
  • a7e2ac7 Adding auto deletion of obj files to build_library
  • a79bab6 modifying the test code for 5 samples

📊 Changes

15 files changed (+5276 additions, -65 deletions)

View changed files

📝 .gitignore (+1 -0)
linux/api_testing.py (+11 -0)
📝 linux/build (+1 -1)
linux/build_api (+44 -0)
linux/build_library (+35 -0)
linux/ccextractor.i (+13 -0)
linux/ccextractor.py (+195 -0)
linux/ccextractor_wrap.c (+4811 -0)
linux/first_phase_evaluation_script (+14 -0)
linux/module_generator (+12 -0)
📝 src/ccextractor.c (+93 -63)
src/ccextractor.h (+41 -0)
📝 src/lib_ccx/ccx_common_option.c (+3 -0)
📝 src/lib_ccx/ccx_common_option.h (+2 -0)
📝 src/lib_ccx/params.c (+0 -1)

📄 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.
  • [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.

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 pull request mainly comprises of two major parts:

  • Refactoring of the source code of ccextractor.c and breaking the code into smaller function.
  • Scripts to generate the python module which would be able to handle a single input file at a single go(so far).

Note: to generate the python module, just run the build_library executable.


🔄 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/744 **Author:** [@Diptanshu8](https://github.com/Diptanshu8) **Created:** 6/27/2017 **Status:** ❌ Closed **Base:** `master` ← **Head:** `first_phase_evaluation` --- ### 📝 Commits (10+) - [`d3540cc`](https://github.com/CCExtractor/ccextractor/commit/d3540ccc0acea0cab70970bbda37fd5d530f88c8) Made temp changes to initiate library coding - [`b0a0c92`](https://github.com/CCExtractor/ccextractor/commit/b0a0c92e50bcd107482d1c8bb7e90347ec0c8190) library development done till api_start. Implementation of stop and status function is left. - [`644d265`](https://github.com/CCExtractor/ccextractor/commit/644d26546cda567f14ace39c4552e2a734f65242) Facing VERSION attribute error in generated module - [`f70f34e`](https://github.com/CCExtractor/ccextractor/commit/f70f34e00924dc33568c423d861b5f87a67173ce) pushing the api code - [`9fa2e3e`](https://github.com/CCExtractor/ccextractor/commit/9fa2e3ebb0892694d030d8c51adc21c822db43c2) adding the ccextractorapi.py file - [`dba1d7b`](https://github.com/CCExtractor/ccextractor/commit/dba1d7b6eb03bb58349b6011ff78460202bb3bbb) Debugging the core dumped error - [`751a22f`](https://github.com/CCExtractor/ccextractor/commit/751a22fe685c5509626b318b116f024b84b40102) Resolving merge conflicts in .gitignore - [`98e295e`](https://github.com/CCExtractor/ccextractor/commit/98e295e76834cdc98cf0b6245dda6178088b97d7) Commiting the build_library file - [`a7e2ac7`](https://github.com/CCExtractor/ccextractor/commit/a7e2ac76868a81117e1ced7d3261e26bd8d27139) Adding auto deletion of obj files to build_library - [`a79bab6`](https://github.com/CCExtractor/ccextractor/commit/a79bab670f1cc55c2e13d94a1f9a54a44aef5d5e) modifying the test code for 5 samples ### 📊 Changes **15 files changed** (+5276 additions, -65 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -0) ➕ `linux/api_testing.py` (+11 -0) 📝 `linux/build` (+1 -1) ➕ `linux/build_api` (+44 -0) ➕ `linux/build_library` (+35 -0) ➕ `linux/ccextractor.i` (+13 -0) ➕ `linux/ccextractor.py` (+195 -0) ➕ `linux/ccextractor_wrap.c` (+4811 -0) ➕ `linux/first_phase_evaluation_script` (+14 -0) ➕ `linux/module_generator` (+12 -0) 📝 `src/ccextractor.c` (+93 -63) ➕ `src/ccextractor.h` (+41 -0) 📝 `src/lib_ccx/ccx_common_option.c` (+3 -0) 📝 `src/lib_ccx/ccx_common_option.h` (+2 -0) 📝 `src/lib_ccx/params.c` (+0 -1) </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. **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 pull request mainly comprises of two major parts: - Refactoring of the source code of `ccextractor.c` and breaking the code into smaller function. - Scripts to generate the python module which would be able to handle a single input file at a single go(so far). Note: to generate the python module, just run the `build_library` executable. --- <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:17:15 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#1565