Minor optimizations by avoiding divisions.

* src/sfnt/ttkern.c (tt_face_load_kern, tt_face_get_kerning):
Replace divisions with multiplication in comparisons.
This commit is contained in:
Werner Lemberg 2010-06-30 10:26:48 +02:00
parent ae425e5189
commit 0ae6cf214f
2 changed files with 10 additions and 3 deletions

View File

@ -1,3 +1,10 @@
2010-06-30 Alexei Podtelezhnikov <apodtele@gmail.com>
Minor optimizations by avoiding divisions.
* src/sfnt/ttkern.c (tt_face_load_kern, tt_face_get_kerning):
Replace divisions with multiplication in comparisons.
2010-06-29 Werner Lemberg <wl@gnu.org> 2010-06-29 Werner Lemberg <wl@gnu.org>
Fix minor tracing issues. Fix minor tracing issues.

View File

@ -5,7 +5,7 @@
/* Load the basic TrueType kerning table. This doesn't handle */ /* Load the basic TrueType kerning table. This doesn't handle */
/* kerning data within the GPOS table at the moment. */ /* kerning data within the GPOS table at the moment. */
/* */ /* */
/* Copyright 1996-2001, 2002, 2003, 2004, 2005, 2006, 2007, 2009 by */ /* Copyright 1996-2001, 2002, 2003, 2004, 2005, 2006, 2007, 2009, 2010 by */
/* David Turner, Robert Wilhelm, and Werner Lemberg. */ /* David Turner, Robert Wilhelm, and Werner Lemberg. */
/* */ /* */
/* This file is part of the FreeType project, and may only be used, */ /* This file is part of the FreeType project, and may only be used, */
@ -115,7 +115,7 @@
num_pairs = FT_NEXT_USHORT( p ); num_pairs = FT_NEXT_USHORT( p );
p += 6; p += 6;
if ( ( p_next - p ) / 6 < (int)num_pairs ) /* handle broken count */ if ( ( p_next - p ) < 6 * (int)num_pairs ) /* handle broken count */
num_pairs = (FT_UInt)( ( p_next - p ) / 6 ); num_pairs = (FT_UInt)( ( p_next - p ) / 6 );
avail |= mask; avail |= mask;
@ -220,7 +220,7 @@
num_pairs = FT_NEXT_USHORT( p ); num_pairs = FT_NEXT_USHORT( p );
p += 6; p += 6;
if ( ( next - p ) / 6 < (int)num_pairs ) /* handle broken count */ if ( ( next - p ) < 6 * (int)num_pairs ) /* handle broken count */
num_pairs = (FT_UInt)( ( next - p ) / 6 ); num_pairs = (FT_UInt)( ( next - p ) / 6 );
switch ( coverage >> 8 ) switch ( coverage >> 8 )