Bug #132
Font$ is messed up in Amiga mode (several problems, but all related)
0%
Description
Font$(x) should contain:
Chars 1-30: Font name (left justified, spaces padded right)
Chars 31-34: Font size (left justified, spaces padded right)
Chars 35-38: Font type (left justified, spaces padded right)
Problems:
- Len(Font$(x)) should always be 38, but it's 39. (I'm guessing that the name has an extra space padded.)
- There are 2 spaces at the beginning of the font size.
- There are 2 spaces at the beginning of the font type.
- The '.font' is not included in the font name.
- The first letter of "rom" should be capitalized. (Rom)
- There should be a space padded after Rom, like so: "Rom ".
- The wrong fonts are coming up in the wrong order, with the wrong pixel heights.
Example:
#manifest: "amiga" Screen Open 0,640,200,32,Hires Get Fonts For f=1 To 9 F$=Font$(f) FontName$=Mid$(F$,1,30) : Rem 1st 30 characters should be the font name FontSize$=Mid$(F$,31,4) : Rem Characters 31-34 should be the font size FontType$=Mid$(F$,35,4) : Rem Characters 35-38 should be the font type Print "Font$(";Right$(Str$(f),Len(Str$(f))-1);"):[";F$;"] (len: ";Len(F$);") should be 38." Print "Font: [";FontName$;"] "; Print "Size: [";FontSize$;"] "; Print "Type: [";FontType$;"] " Next f
There should be 9 ROM fonts (1-9), in the following order, with the pixel heights shown below:
1: topaz.font, 8 2: topaz.font, 9 3: 2001.font, 8 4: broadway.font, 16 5: lines.font, 21 6: mFast.font, 20 7: News.font, 23 8: Peridot.font, 7 9: Pica.font, 32
Instead, now we're getting:
1. broadway, 16 2. lines, 16 3. mFast, 16 4. News, 16 5. peridot, 16 6. Pica, 16 7. sapphire, 16 8. n/a 9. n/a
Updated by Brian Flanagan over 3 years ago
- Affected version changed from 0.9.4 to 0.9.5.1
Problems still exist in 0.9.5.1.
Updated by Brian Flanagan almost 3 years ago
- Affected version changed from 0.9.5.1 to 0.9.9.3
Problem still exists in 0.9.9.3
In addition, now Get Fonts, Get ROM Fonts, Get Disk Fonts all fail.
Updated by Brian Flanagan almost 3 years ago
- Affected version changed from 0.9.9.3 to 0.9.9.4
Problems still exist. (0.9.9.4 rel1)
Updated by Brian Flanagan over 2 years ago
- Estimated time set to 2:00 h
- Affected version changed from 0.9.9.4 to Beta RC3
We're now getting 9 fonts showing in the Amiga manifest, but they're still the wrong face & name.
0 2001 16 rom 1 broadway 16 rom 2 lines 16 rom 3 mFast 16 rom 4 News 16 rom 5 peridot 16 rom 6 Pica 16 rom 7 sapphire 16 rom 8 topaz 16 rom
Updated by Brian Flanagan over 2 years ago
that last comment should have read "wrong face & size"
Updated by Brian Flanagan over 2 years ago
- Affected version changed from Beta RC3 to 1.0.0 (B3)
Font$ is still pretty messed up in Amiga mode. The above problems still exist.
Summary:
The default font is wrong.
The fonts faces and sizes are wrong.
Referencing Font$(n) where n is > # of fonts -1 causes an Internal error.
Font$(0) should be an empty string.
The string returned by Font$ is incorrectly formatted. It should be:
Bytes 1-30: Font Name ( left justified, spaces padded right)
Bytes 31-34: Font Size ( left justified, spaces padded right)
Bytes 35-38: Font Type ( left justified, spaces padded right) Contains "Rom " or "Disc"
The Length of Font$(n) should ALWAYS be 38 (with format specified as above)