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

Wrapped libtiff5 and libltdl7 #1677

Merged
merged 7 commits into from
Jul 15, 2024
Merged

Wrapped libtiff5 and libltdl7 #1677

merged 7 commits into from
Jul 15, 2024

Conversation

kaixindeken
Copy link
Contributor

@kaixindeken kaixindeken commented Jul 15, 2024

  • wrapped libtiff5

  • wrapped libltdl7

@kaixindeken kaixindeken changed the title Wrapped libtiff5 Wrapped libtiff5 and libltdl7 Jul 15, 2024
@kaixindeken
Copy link
Contributor Author

Still in Progress

@kaixindeken
Copy link
Contributor Author

Done

@ptitSeb ptitSeb merged commit 94ccde1 into ptitSeb:main Jul 15, 2024
47 checks passed
@kaixindeken
Copy link
Contributor Author

Looking good while rendering the PDF version of box86.com

2024-07-15_14-24-24

@ptitSeb
Copy link
Owner

ptitSeb commented Jul 15, 2024

On what architecture is that screenshot? Arm? RV64 or LA64 (or something else)?

@kaixindeken
Copy link
Contributor Author

kaixindeken commented Jul 15, 2024

On what architecture is that screenshot? Arm? RV64 or LA64 (or something else)?

on rv64

2024-07-15_14-53-17

GO(TIFFInitZSTD, iFpi)
GO(TIFFInitWebP, iFpi)
// GO(TIFFCIELabToXYZ, vFpuiippp)
GO(TIFFXYZToRGB, iFpppp)
Copy link
Collaborator

Choose a reason for hiding this comment

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

void TIFFXYZToRGB(TIFFCIELabToRGB *cielab, float X, float Y, float Z, uint32_t *R, uint32_t *G, uint32_t *B)

I’m on my phone, so I randomly picked a function to check, and it seems badly wrong?

Copy link
Owner

Choose a reason for hiding this comment

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

Ok, I concur, this function has a bad signature.

@kaixindeken how are you making the function signatures? Should I revert all your PR?

Copy link
Owner

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

my apologis, i'll fix it right now. please do not revert this pr. i'll create a new pr to fix it all.

@ksco
Copy link
Collaborator

ksco commented Jul 15, 2024

If you don’t understand how libwrapping works, feel free to talk to me first instead of submit something that is very wrong.

Javier97sm pushed a commit to Javier97sm/box64 that referenced this pull request Aug 16, 2024
* Wrapped libtiff5

* no types generated

* libtiff5 added missing defs types undefs files

* wrapped libltdl7

* rerun actions

* add more wrapped function for tiff5 and ltdl7

* remove unused functions
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