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

Update FreeType to v2.13.2 #431

Merged
merged 8 commits into from
Jan 13, 2024
Merged

Conversation

a740g
Copy link
Contributor

@a740g a740g commented Jan 7, 2024

This PR helps transition from the outdated FreeTypeAmalgam library to the latest supported FreeType library for enhanced bug fixes, security updates, and improvements. FreeTypeAmalgam is an amalgamation of the older FreeType library. FreeType has undergone various bug fixes, security updates, and improvements over the years.

  • Benefits:

    • Integration of the latest FreeType bug fixes and security updates.
    • Incorporation of various FreeType improvements.
  • Key Changes:

    • Adjustments in internal/c/parts/video/font/build.mk for FreeType flat-directory build logic.
    • Inclusion of a flattened local copy of the FreeType source tree.

@a740g a740g added the enhancement New feature or request label Jan 7, 2024
@a740g a740g self-assigned this Jan 7, 2024
Copy link
Member

@RhoSigma-QB64 RhoSigma-QB64 left a comment

Choose a reason for hiding this comment

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

Assuming you know what you do, not in the mood to deeply check into 400+ changed files :)

@a740g
Copy link
Contributor Author

a740g commented Jan 9, 2024

Assuming you know what you do, not in the mood to deeply check into 400+ changed files :)

😁 Those 400+ files all belong to FreeType. Code wise, I only changed 3 lines of C (of what we already had) in font.cpp. The remaining changes are all Makefile black-magic I picked up from @mkilgore.

@SteveMcNeill
Copy link
Member

Assuming you know what you do, not in the mood to deeply check into 400+ changed files :)

I only changed 3 lines of C (of what we already had) in font.cpp. The remaining changes are all Makefile black-magic I picked up from @mkilgore.

Ah ha! Now I know who to blame when those 3 lines turn out to be written backwards! ;)

@a740g a740g merged commit 4c820fb into QB64-Phoenix-Edition:main Jan 13, 2024
4 checks passed
@a740g a740g deleted the freetype-upgrade branch January 13, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants