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

MµPDF 1.14.0 #762

Closed
wants to merge 23 commits into from
Closed

MµPDF 1.14.0 #762

wants to merge 23 commits into from

Conversation

NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Nov 14, 2018

Here be dragons, a few API changes to pass some stuff by value instead of by reference...

And a buildsystem update, because of course. :D.


This change is Reviewable

@NiLuJe
Copy link
Member Author

NiLuJe commented Nov 14, 2018

Special shout out to past self in #660 ;p.

@NiLuJe
Copy link
Member Author

NiLuJe commented Nov 14, 2018

Sidebar: upstream's documentation is still abysmal, ripgrep is my best friend. As is landing by chance on the right commit message.

Which may very well mean that shit I may have missed will break.

And, let's face it, there's a 142% probability that I did miss something ;).

@NiLuJe
Copy link
Member Author

NiLuJe commented Nov 15, 2018

Library authors: for the love of all that is holy, please document API changes in a clear, concise, and easy to find way.

Because it appears to be including hb.h even with HB disabled, or
something.
Copy link
Member Author

@NiLuJe NiLuJe left a comment

Choose a reason for hiding this comment

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

This makes everything fantastically opaque, not a fan, but hey. -_-".

@NiLuJe

This comment has been minimized.

@NiLuJe

This comment has been minimized.

@NiLuJe
Copy link
Member Author

NiLuJe commented Nov 15, 2018

Pro tip: calling a function that doesn't exist will segfault! :D.

pdf_set_markup_appearance is gone.

@NiLuJe
Copy link
Member Author

NiLuJe commented Nov 15, 2018

I have no idea how any of this is supposed to work, so, if anyone wants to take care of it, be my guest ;).

@NiLuJe
Copy link
Member Author

NiLuJe commented Nov 16, 2018 via email

@NiLuJe
Copy link
Member Author

NiLuJe commented Nov 18, 2018

(There's a link to the offending commit for the current issue in a comment added in the latest commit).

@Frenzie
Copy link
Member

Frenzie commented Nov 18, 2018

Good, that's one step closer. ;-)

@baskerville
Copy link
Contributor

The following command might be helpful in knowing precisely how the API changed:

git diff 1.13.0..1.14.0 include

@NiLuJe
Copy link
Member Author

NiLuJe commented Dec 23, 2018

My main concern is no longer ranting about undocumented changes, but instead figuring out the replacing API with zero knowledge of the PDF format :D.

@bahusoid
Copy link

bahusoid commented May 8, 2019

for the love of all that is holy, please document API changes in a clear, concise, and easy to find way.

And voila MuPdf 1.15 is out with API changes :) At least made in 1.14

Regarding replacing API for pdf_set_markup_appearance.

Have you looked here (see how this new_annot is used)? I might be wrong but seems it contains similar logic that creates and manipulates with annotations. Looks like you need to simply call pdf_update_appearance instead plus:

At the end of manipulations it seems second call to pdf_update_appearance is required.

@NiLuJe
Copy link
Member Author

NiLuJe commented May 8, 2019

Thanks for the heads up ;). That looks sensible, thanks!

This'll have to wait until k2pdfopt moves to this new MuPDF version, though ;). Provided the accompanying bump to tesseract and friends doesn't also drive me mad ;p.

@Frenzie
Copy link
Member

Frenzie commented Jul 29, 2019

@NiLuJe
Copy link
Member Author

NiLuJe commented Jul 29, 2019 via email

@Frenzie
Copy link
Member

Frenzie commented Jun 21, 2020

@NiLuJe I hope I merged it right; it was a quickie using the GH web GUI. k2pdfopt 2.52 just came out but it's still on MuPDF 1.14 I believe.

@NiLuJe
Copy link
Member Author

NiLuJe commented Jun 21, 2020

I'll take a look at it sometime this week ;).

@roshavagarga
Copy link

@Frenzie @NiLuJe k2pdfopt 2.53 seems to have updated to MuPDF 1.17.

@NiLuJe
Copy link
Member Author

NiLuJe commented Jul 26, 2020

@roshavagarga: Ah, now we're talking, thanks for the heads up!

I'll try to take a look at it Soon(TM) ;).

@NiLuJe
Copy link
Member Author

NiLuJe commented Aug 1, 2020

Sidebar: soon may be late, because I won't be available for most of August ;p.

@Frenzie
Copy link
Member

Frenzie commented Aug 1, 2020

Well, have fun! ;-)

@Frenzie
Copy link
Member

Frenzie commented Dec 24, 2020

Superseded by #1203.

@Frenzie Frenzie closed this Dec 24, 2020
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.

5 participants