Skip to content

Fix extra PDFDoc() events in SWIG wrappers#212

Open
myang-apryse wants to merge 1 commit into
next_releasefrom
SVR-11475-Fix-SWIG-wrappers-in-PDFNetWrappers-that-caused-extra-PDFDoc-events
Open

Fix extra PDFDoc() events in SWIG wrappers#212
myang-apryse wants to merge 1 commit into
next_releasefrom
SVR-11475-Fix-SWIG-wrappers-in-PDFNetWrappers-that-caused-extra-PDFDoc-events

Conversation

@myang-apryse
Copy link
Copy Markdown
Contributor

@myang-apryse myang-apryse commented Jun 3, 2026

Summary

Port changes from PR #180

Context/Why?

Previous PR only handled Ruby, but it appears to have correctly fixed it.
We're now porting over the fix to the other SWIG languages

Implementation notes

How to verify

  • Using PWSTester:
    • latest PWSTester requires the ScopeBegin/End changes from sch/ftr-3165
    • either rebase them onto master or build directly from this branch
    • after running, check test_output/stats/per_api_report and compare with previous

https://jenkins.apryse.com/job/apryse-sdk/job/apryse-sdk-go-windows/view/change-requests/job/PR-212/
https://jenkins.apryse.com/job/apryse-sdk/job/apryse-sdk-ruby-alpine/view/change-requests/job/PR-6432/
https://jenkins.apryse.com/job/apryse-sdk/job/apryse-sdk-php-linux/view/change-requests/job/PR-6432/
https://jenkins.apryse.com/job/apryse-sdk/job/apryse-sdk-py3-windows/view/change-requests/job/PR-6432/

Changelog entry

N/A

@myang-apryse myang-apryse marked this pull request as ready for review June 3, 2026 22:56
@myang-apryse myang-apryse changed the base branch from master to next_release June 3, 2026 23:18
Copy link
Copy Markdown
Contributor

@overocean overocean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess you need to create a branch from next_release. The diff doesn't look right.

@myang-apryse myang-apryse force-pushed the SVR-11475-Fix-SWIG-wrappers-in-PDFNetWrappers-that-caused-extra-PDFDoc-events branch from e8f7078 to 0407f43 Compare June 4, 2026 16:46
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.

3 participants