Re: wsfont.c: remove sony8x16 and sony12x24?

[Frederic Cambus]

On Thu, Jan 12, 2017 at 01:39:47PM +0100, Frederic Cambus wrote:

> Most fonts in wsfont are not compiled in by default, and while some are
> worth keeping and using, I would like to propose deleting some of them
> in order to make space for larger sized fonts.
> 
> The Sony fonts are Serif, which doesn't work very well for monospaced
> bitmap fonts intended for console use, are not very readable, and have
> no slashed or dotted zeros. Delete them?
> 
> For obvious reasons, I didn't run cvs rm on sony8x16.h and sony12x24.h
> before generating the diff.
> 
> Comments? OK?

It's been five days and so far nobody objected. Got one OK privately,
however I would prefer having more feedback before committing this.

If there are no objections I'm going to commit in the following days.

Any more OKs? Any objections?

wsfont.c: remove sony8x16 and sony12x24?

[Frederic Cambus]

Hi tech@,

Most fonts in wsfont are not compiled in by default, and while some are
worth keeping and using, I would like to propose deleting some of them
in order to make space for larger sized fonts.

The Sony fonts are Serif, which doesn't work very well for monospaced
bitmap fonts intended for console use, are not very readable, and have
no slashed or dotted zeros. Delete them?

For obvious reasons, I didn't run cvs rm on sony8x16.h and sony12x24.h
before generating the diff.

Comments? OK?

Index: distrib/sets/lists/comp/mi
===
RCS file: /cvs/src/distrib/sets/lists/comp/mi,v
retrieving revision 1.1322
diff -u -p -r1.1322 mi
--- distrib/sets/lists/comp/mi  7 Jan 2017 16:04:13 -   1.1322
+++ distrib/sets/lists/comp/mi  11 Jan 2017 21:45:55 -
@@ -813,8 +813,6 @@
 ./usr/include/dev/wsfont/lucida16x29.h
 ./usr/include/dev/wsfont/omron12x20.h
 ./usr/include/dev/wsfont/qvss8x15.h
-./usr/include/dev/wsfont/sony12x24.h
-./usr/include/dev/wsfont/sony8x16.h
 ./usr/include/dev/wsfont/vt220l8x10.h
 ./usr/include/dev/wsfont/vt220l8x8.h
 ./usr/include/dev/wsfont/wsfont.h
Index: sys/dev/wsfont/wsfont.c
===
RCS file: /cvs/src/sys/dev/wsfont/wsfont.c,v
retrieving revision 1.45
diff -u -p -r1.45 wsfont.c
--- sys/dev/wsfont/wsfont.c 6 Oct 2016 07:30:19 -   1.45
+++ sys/dev/wsfont/wsfont.c 11 Jan 2017 21:45:55 -
@@ -64,16 +64,6 @@
 #include 
 #endif
 
-#ifdef FONT_SONY8x16
-#define HAVE_FONT 1
-#include 
-#endif
-
-#ifdef FONT_SONY12x24
-#define HAVE_FONT 1
-#include 
-#endif
-
 #ifdef FONT_OMRON12x20
 #define HAVE_FONT 1
 #include 
@@ -154,14 +144,8 @@ static struct font builtin_fonts[] = {
 #ifdef FONT_VT220L8x10
BUILTIN_FONT(vt220l8x10, 7),
 #endif
-#ifdef FONT_SONY8x16
-   BUILTIN_FONT(sony8x16, 8),
-#endif
-#ifdef FONT_SONY12x24
-   BUILTIN_FONT(sony12x24, 9),
-#endif
 #ifdef FONT_OMRON12x20
-   BUILTIN_FONT(omron12x20, 10),
+   BUILTIN_FONT(omron12x20, 8),
 #endif
 #undef BUILTIN_FONT
 };