[Git][freetype/freetype][master] 2 commits: [bdf, pcf] Minor optimizatio

Alexei Podtelezhnikov pushed to branch master at FreeType / FreeType

Commits:

90b14882

by Alexei Podtelezhnikov at 2021-09-22T20:20:04-04:00

[bdf, pcf] Minor optimization.

* src/pcf/pcfread.c (pcf_load_font): Do not call `FT_MulDiv` for a
small job.
* src/bdf/bdfdrivr.c (BDF_Face_Init): Ditto.
* src/bdf/bdflib.c (_bdf_parse_glyphs): Fix a comment.

3b573e45
by Alexei Podtelezhnikov at 2021-09-22T22:22:13-04:00
```
Reference `fopen` in the docs.
```

Changes:

include/freetype/freetype.h

@@ -2186,6 +2186,10 @@ FT_BEGIN_HEADER
     *   FreeType error code.  0~means success.
+    *
     * @note:
 +   *   The `pathname` string should be reconizable as such by a standard
 +   *   `fopen()` call on your system.  If that is not sufficient to address
 +   *   all file name possiblities, you can use @FT_Open_Face.
 +   *
     *   Use @FT_Done_Face to destroy the created @FT_Face object (along with
     *   its slot and sizes).
     */

src/bdf/bdfdrivr.c


         else
         {
           /* this is a heuristical value */
-          bsize->width = (FT_Short)FT_MulDiv( bsize->height, 2, 3 );
+          bsize->width = ( bsize->height * 2 + 1 ) / 3;
         }
 
         prop = bdf_get_font_property( font, "POINT_SIZE" );

src/bdf/bdflib.c


       goto Exit;
     }
 
-    /* Expect the DWIDTH (scalable width) field next. */
+    /* Expect the DWIDTH (device width) field next. */
     if ( _bdf_strncmp( line, "DWIDTH", 6 ) == 0 )
     {
       error = _bdf_list_split( &p->list, " +", line, linelen );

src/pcf/pcfread.c


         else
         {
           /* this is a heuristical value */
-          bsize->width = (FT_Short)FT_MulDiv( bsize->height, 2, 3 );
+          bsize->width = ( bsize->height * 2 + 1 ) / 3;
         }
 
         prop = pcf_find_property( face, "POINT_SIZE" );

From:	Alexei Podtelezhnikov (@apodtele)
Subject:	[Git][freetype/freetype][master] 2 commits: [bdf, pcf] Minor optimization.
Date:	Thu, 23 Sep 2021 02:24:21 +0000

...	...	@@ -487,7 +487,7 @@ THE SOFTWARE.
487	487	else
488	488	{
489	489	/* this is a heuristical value */
490		- bsize->width = (FT_Short)FT_MulDiv( bsize->height, 2, 3 );
	490	+ bsize->width = ( bsize->height * 2 + 1 ) / 3;
491	491	}
492	492
493	493	prop = bdf_get_font_property( font, "POINT_SIZE" );

...	...	@@ -1615,7 +1615,7 @@
1615	1615	goto Exit;
1616	1616	}
1617	1617
1618		- /* Expect the DWIDTH (scalable width) field next. */
	1618	+ /* Expect the DWIDTH (device width) field next. */
1619	1619	if ( _bdf_strncmp( line, "DWIDTH", 6 ) == 0 )
1620	1620	{
1621	1621	error = _bdf_list_split( &p->list, " +", line, linelen );

...	...	@@ -1612,7 +1612,7 @@ THE SOFTWARE.
1612	1612	else
1613	1613	{
1614	1614	/* this is a heuristical value */
1615		- bsize->width = (FT_Short)FT_MulDiv( bsize->height, 2, 3 );
	1615	+ bsize->width = ( bsize->height * 2 + 1 ) / 3;
1616	1616	}
1617	1617
1618	1618	prop = pcf_find_property( face, "POINT_SIZE" );