Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Leptonica 1.83.0 #54

Merged
merged 4 commits into from
Feb 21, 2023
Merged

Leptonica 1.83.0 #54

merged 4 commits into from
Feb 21, 2023

Conversation

ccouzens
Copy link
Collaborator

Make changes to support leptonica 1.83.0

ccouzens/leptonica-plumbing#5

Basically leptonica made the fields within its structs private. Which means we can only get go them via leptonica's methods.

In the case of pixa and boxa, it required bigger changes as we could no longer get to their items directly. When going through leptonica's methods, we no longer merely borrow the memory, but get a reference counted copy. This requires us to handle it differently (eg dropping it afterwards).

ccouzens/leptonica-plumbing#5

It doesn't currently work, but I expect it can be refactored.

```bash
LD_LIBRARY_PATH="$(pwd)/../../DanBloomberg/leptonica/local/lib" PKG_CONFIG_PATH="$(pwd)/../../DanBloomberg/leptonica/local/lib/pkgconfig" bash -c 'cargo test --release && valgrind --leak-check=yes --error-exitcode=1 --leak-check=full --show-leak-kinds=all "$(find target/*/deps/ -executable -name 'leptess-*')"'
```
@ccouzens ccouzens requested a review from houqp February 19, 2023 21:15
@ccouzens ccouzens linked an issue Feb 19, 2023 that may be closed by this pull request
Seeing as it will be a new release.

It is technically a breaking change in parts, because the exposed leptonica-plumbing library has a new interface.
And because `set_rectangle` has become `set_rectangle_from_box` , and some return types have changed.
@ccouzens ccouzens merged commit 3ba68d3 into master Feb 21, 2023
@ccouzens ccouzens deleted the leptonica_1.83.0 branch February 21, 2023 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

leptonica v1.83.0
1 participant