fix: Code reorg - #156

Merged

tmathern merged 14 commits into

mainfrom

mathern/code-reorg

Sep 3, 2025

tmathern commented Aug 29, 2025 •

edited

Loading

Collaborator

Reorganize the code (method order in the classes)
One improvement for memory handling in Reader
Some minor improvements to avoid repeated lookup on self / possible repeated calls

tmathern added 11 commits

August 29, 2025 12:30


          fix: Builder state

9534fd9


          fix: reader state

618fd42


          fix: Clean up states

519282a


          fix: Comment clean up

f08eea2


          fix: Microoptimization


          fix: One less copy

7a91226


          fix: Clean up comment

34f20d3


          fix: Reorg code

28b4e5f


          fix: Remove a prop

48cfbd2


          fix: Small change

1940d9a


          fix: Clean up comments

6d9ab7d

tmathern requested review from dyro, gpeacock and ok-nick

August 29, 2025 21:08

tmathern self-assigned this

tmathern marked this pull request as draft

August 29, 2025 21:09


          fix: Refactor more

5359bcf

tmathern commented

View reviewed changes

src/c2pa/c2pa.py

tmathern commented

View reviewed changes

src/c2pa/c2pa.py

tmathern commented

View reviewed changes

src/c2pa/c2pa.py

tmathern commented

View reviewed changes

src/c2pa/c2pa.py

tmathern commented

View reviewed changes

src/c2pa/c2pa.py

tmathern commented

View reviewed changes

src/c2pa/c2pa.py

tmathern commented

View reviewed changes

src/c2pa/c2pa.py

tmathern marked this pull request as ready for review

August 29, 2025 21:15

tmathern changed the title ~~fix: Code reorg and (micro)optimizations~~ fix: Code reorg

gpeacock approved these changes

View reviewed changes

Base automatically changed from mathern/init-states to main

September 3, 2025 19:47

tmathern added 2 commits

September 3, 2025 12:48


          fix: Merge in latest main

596dfad


          fix: Code reorg merging conflicts

c639d7f

tmathern merged commit 78b14ac into main

20 checks passed

tmathern deleted the mathern/code-reorg branch

September 3, 2025 20:01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet