Skip to content

READ-BDF: A few fixes and performance improvements - #2519

Draft
MattHeffron wants to merge 17 commits into
masterfrom
mth63--Misc_READ-BDF_fixes_and_performance_changes
Draft

READ-BDF: A few fixes and performance improvements#2519
MattHeffron wants to merge 17 commits into
masterfrom
mth63--Misc_READ-BDF_fixes_and_performance_changes

Conversation

@MattHeffron

Copy link
Copy Markdown
Member

A few fixes and performance improvements.
Added a parameter to help testing the apparent memory leak issue.

@MattHeffron
MattHeffron marked this pull request as draft February 25, 2026 08:02
@pamoroso

Copy link
Copy Markdown
Member

On Linux Mint 22.1 Cinnamon I get a filesystem protection violation error when calling WRITE-BDF-TO-MEDLEYDISPLAYFONT-FILE:

fonterr

The fonts directory seems to have the right permissions:

drwxrwxr-x 4 paolo paolo   4096 Feb 25 12:18 fonts

What am I doing wrong?

@MattHeffron

Copy link
Copy Markdown
Member Author

What am I doing wrong?

Your {DSK}<fonts> directory corresponds to /fonts/, i.e., a root level directory. Is that actually correct?
Can you create any other file there? For example, (DRIBBLE "{DSK}<fonts>test.dribble")
WRITE-BDF-TO-MEDLEYDISPLAYFONT-FILE and MEDLEY.WRITE.FONT don't do anything special with the destination.

@rmkaplan

rmkaplan commented Feb 27, 2026 via email

Copy link
Copy Markdown
Contributor

@MattHeffron

Copy link
Copy Markdown
Member Author

Do I really have {DSK} ?

@pamoroso has {DSK} in the call to WRITE-BDF-TO-MEDLEYDISPLAYFONT-FILE which constructs the full path for MEDLEY.WRITE.FONT

@pamoroso

Copy link
Copy Markdown
Member

My fonts directory is ~/il/fonts/ and I have this in my init:

(SETQ DISPLAYFONTDIRECTORIES (APPEND '(NIL "{DSK}<home>paolo>il>fonts>Commodore>" 
                                                      "{DSK}<home>paolo>il>fonts>ComicSans>" 
                                                      "{DSK}<home>paolo>il>fonts>")
                                               DISPLAYFONTDIRECTORIES))

(DRIBBLE "{DSK}<fonts>test.dribble") yields the error Protection violation: {DSK}<fonts>test.dribble;1, so it looks like I'm trying to write outside of my home directory.

@pamoroso

Copy link
Copy Markdown
Member

At commit 8474e63 nothing unusual to report on Linux Mint 22.1 Cinnamon.

@pamoroso

Copy link
Copy Markdown
Member

I updated to commit 7b2b6ef and tried to call READ-BDF on a font downloaded from here. I renamed the font file to test.bdf, copied it to my Medley user directory, and called READ-BDF but got this error:

5_ (BDF:READ-BDF 'test.bdf)
In IL:ERROR:
ISO8859/1 is not an external format

Is this expected?

@rmkaplan

rmkaplan commented Apr 26, 2026 via email

Copy link
Copy Markdown
Contributor

@masinter

Copy link
Copy Markdown
Member

the preferred IANA name is ISO-8859-1 and I think we should follow IANA registry with keywords matching when possible. No /.

https://www.iana.org/assignments/character-sets

@pamoroso

pamoroso commented May 9, 2026

Copy link
Copy Markdown
Member

Nothing to report at commit 3f8e935 other than the persistance of the external format error.

@rmkaplan

rmkaplan commented May 9, 2026 via email

Copy link
Copy Markdown
Contributor

@rmkaplan

rmkaplan commented May 9, 2026

Copy link
Copy Markdown
Contributor

But if it really wants ISO8859IO/1, try first (FILESLOAD ISO8859IO)

@pamoroso

pamoroso commented May 9, 2026

Copy link
Copy Markdown
Member

After (FILESLOAD ISO8859IO) the call (BDF:READ-BDF 'test.bdf) completes with no errors.

@pamoroso

Copy link
Copy Markdown
Member

Still nothing unusual at commit d86e0d3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants