/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.apache.fontbox.ttf;

import java.io.IOException;

/**
 * A table in a true type font.
 * 
 * @author Ben Litchfield (ben@benlitchfield.com)
 * @version $Revision: 1.1 $
 */
public class GlyphTable extends TTFTable
{
    /**
     * Tag to identify this table.
     */
    public static final String TAG = "glyf";
    
    private GlyphData[] glyphs;
    
    /**
     * This will read the required data from the stream.
     * 
     * @param ttf The font that is being read.
     * @param data The stream to read the data from.
     * @throws IOException If there is an error reading the data.
     */
    public void initData( TrueTypeFont ttf, TTFDataStream data ) throws IOException
    {
        MaximumProfileTable maxp = ttf.getMaximumProfile();
        IndexToLocationTable loc = ttf.getIndexToLocation();
        // the glyph offsets
        long[] offsets = loc.getOffsets();
        // number of glyphs
        int numGlyphs = maxp.getNumGlyphs();
        // the end of the glyph table
        long endOfGlyphs = offsets[numGlyphs];
        long currentOffset = -1;
        long offset = getOffset();
        glyphs = new GlyphData[numGlyphs];
        for( int i=0; i<numGlyphs; i++ )
        {
            // end of glyphs reached?
            if (endOfGlyphs == offsets[i])
            {
                break;
            }
            // don't repeat glyphs
            if (currentOffset == offsets[i])
            {
                continue;
            }
            currentOffset = offsets[i];
            glyphs[i] = new GlyphData();
            data.seek( offset + offsets[i] );
            glyphs[i].initData( ttf, data );
        }
        for( int i=0; i<numGlyphs; i++ )
        {
            GlyphData glyph = glyphs[i];
            // resolve composite glyphs
            if (glyph != null && glyph.getDescription().isComposite())
            {
                glyph.getDescription().resolve();
            }
        }
    }
    
    /**
     * @return Returns the glyphs.
     */
    public GlyphData[] getGlyphs()
    {
        return glyphs;
    }
    
    /**
     * @param glyphsValue The glyphs to set.
     */
    public void setGlyphs(GlyphData[] glyphsValue)
    {
        glyphs = glyphsValue;
    }
}

@@ -955,33 +955,8 @@
 %0A * 
-@version $Revision: 1.1 $
 %0A */
@@ -1095,20 +1095,16 @@
 %22glyf%22;%0A
-    
 %0A    pri
@@ -1128,20 +1128,16 @@
 glyphs;%0A
-    
 %0A    /**
@@ -1401,17 +1401,16 @@
 nitData(
- 
 TrueType
@@ -1437,17 +1437,16 @@
 eam data
- 
 ) throws
@@ -1819,41 +1819,8 @@
 s%5D;%0A
-        long currentOffset = -1;%0A
     
@@ -1900,37 +1900,41 @@
 %0A        for
-(
  
+(
 int i
-=
+ = 
 0; i
-%3C
+ %3C 
 numGlyphs; i
@@ -1927,33 +1927,32 @@
 %3C numGlyphs; i++
- 
 )%0A        %7B%0A    
@@ -2098,57 +2098,125 @@
  // 
-don't repeat glyphs%0A            if (currentO
+the current glyph isn't defined%0A            // if the next offset equals the current index%0A            if (o
 ffset
+s%5Bi%5D
  == 
@@ -2216,32 +2216,36 @@
 %5Bi%5D == offsets%5Bi
+ + 1
 %5D)%0A            %7B
@@ -2289,48 +2289,8 @@
   %7D%0A
-            currentOffset = offsets%5Bi%5D;%0A
     
@@ -2348,17 +2348,16 @@
 ta.seek(
- 
 offset +
@@ -2367,17 +2367,16 @@
 fsets%5Bi%5D
- 
 );%0A     
@@ -2401,17 +2401,16 @@
 nitData(
- 
 ttf, dat
@@ -2410,17 +2410,16 @@
 tf, data
- 
 );%0A     
@@ -2438,21 +2438,25 @@
  for
-(
  
+(
 int i
-=
+ = 
 0; i
-%3C
+ %3C 
 numG
@@ -2465,17 +2465,16 @@
 phs; i++
- 
 )%0A      
@@ -2719,28 +2719,24 @@
     %7D%0A    %7D%0A
-    
 %0A    /**%0A   
@@ -2845,20 +2845,16 @@
 ;%0A    %7D%0A
-    
 %0A    /**

fix skipped glyph ids