-
-
Notifications
You must be signed in to change notification settings - Fork 29
Complete the READ-BDF changes for MEDLEYDISPLAYFONT format #2395
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
base: master
Are you sure you want to change the base?
Complete the READ-BDF changes for MEDLEYDISPLAYFONT format #2395
Conversation
Various renaming for consistency with XCCS -> MCCS changes. Use IL:FONTSPEC record instead of using FIRST, SECOND, etc. Fix the parsing of IL:FONTSPEC to use COMPRESSED instead of incorrect CONDENSED. Zero-width "image" with zero-width "escapement" GLYPHS get put into NOMAPPINGCHARSET. Add (FILES (SYSLOAD) SYSEDIT) under existing (DECLARE: EVAL@COMPILE DONTCOPY ...)
WRITE-BDF-TO-DISPLAYFONT-FILES is deprecated (but symbols imported from IL: only for use there are not yet removed from :IMPORT-FROM)
Removed WRITE-BDF-TO-DISPLAYFONT-FILE (i.e., no STRIKE format files). No multiple-values returned. Instead, use LIST when appropriate. BDF-TO-CHARSETINFO is now IDEMPOTENT w.r.t. the GLYPHS. Move resolution and defaulting of FAMILY, FACE, SIZE, ROTATION, DEVICE from WRITE-BDF-TO-MEDLEYDISPLAYFONT-FILE to BDF-TO-FONTDESCRIPTOR. Keep the MCCS chars present BITMAP in the BDF-FONT structure, instead of needing to schlepping it around separately. Abstracted testing/setting the MCCS chars present bits to CHAR-PRESENT-BIT (mimicking BITMAPBIT). Added COUNT-MCHARS to know how many MCCS chars are marked in the BITMAP as present. READ-BDF now handles when UTOMCODE? returns multiple mappings, and creates the appropriate duplicate GLYPHS with different MCCS char codes. READ-GLYPH doesn't create an empty BITMAP for spacing glyphs. Use font code changes: Set (CHARSETINFO CHARSETNO). Set (FONTDESCRIPTOR FONTSLUGWIDTH).
|
Can you include one of the medleyfonts that you are now producing? |
|
Here are 2 font files, and PDFs of FontSampler of the fonts. (Zipped so github will accept them.) |
Changed parameters for BUILD-COMPOSITE to simplify, and enable keyword :VERBOSE parameter. Added some VERBOSE progress messages.
|
On the Tedit file: In the notes: it is also unwise to have a hyphen in a family name. Why do you recommend not putting them in one of the DISPLAYFONTDIRECTORIES? Why do you think they would be lost if Medley is updated, if they have been committed? You might clarify that this warning is for the situation when somebody is creating fonts in their own environment, and not committing them to Medley. |
|
The only thing I saw in the font files is that the FONTCHARENCODING property should be MCCS, not e.g. NOTOSANS. Is that coming from \READCHARSET or from your own function? |
masinter
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think moving the old versions of READ-BDF to obsolete/ is a good idea. Old versions are readily available in the git hisstory.
in some cases where we're depricating old software and we're not quite sure of the deprecation it might be OK but that doesn't apply here.
It's not in my code. I don't call \READCHARSET. It looks like it is in FNS ;; Under (SELECTQ
(CHARENCODING [OR (ffetch FONTCHARENCODING of FONT)
(freplace FONTCHARENCODING of FONT
with (if (MEMB (fetch (FONTDESCRIPTOR FONTFAMILY) of FONT)
NSFONTFAMILIES)
then 'XCCS$
elseif (MEMB (fetch (FONTDESCRIPTOR FONTFAMILY) of FONT)
ALTOFONTFAMILIES)
then 'ALTOTEXT
else (fetch (FONTDESCRIPTOR FONTFAMILY) of FONT])that last else is defaulting to the FAMILY. |
(This needs regression tests. "Harmless" edits aren't!) Documentation review changes.
|
@rmkaplan No, I don't use |
Removed. |
|
I updated to commit c25da55 and it's still working with no issues. |
|
I updated to commit d2b1726, still no issues. |
…ONTDESCRIPTOR, respectively, explicitly be MCCS
|
@rmkaplan |

This should complete the changes for the MEDLEYDISPLAYFONTFORMAT.
The building a composite font from BDF files implemented.
Resolves #2313
Resolves #2365
This should make #2364 possible.
(I created a size 20 Noto Sans composite font from the same Google fonts as used by @hjellinek's HTMLSTREAM.)
Added files with version of partial implementation of this for old (STRIKE) file format to
obsolete/lispusers/READ-BDF-old/