Revert "[4.0] Add debug log for glyph positions in text layout engine" 47/169647/1 accepted/tizen/4.0/unified/20180212.141240 submit/tizen_4.0/20180209.045021
authorhuiyu,eun <huiyu.eun@samsung.com>
Thu, 8 Feb 2018 06:02:32 +0000 (15:02 +0900)
committerhuiyu,eun <huiyu.eun@samsung.com>
Thu, 8 Feb 2018 06:02:36 +0000 (15:02 +0900)
This reverts commit a9ce4ade4d1b2e1a87e9fdcb540da4fe1beb0fd9.

Change-Id: Ia5438366f1bfc0d87c9cdef79dd9100cfa4d490a

dali-toolkit/internal/text/layouts/layout-engine.cpp

index e273049..4ace75b 100644 (file)
@@ -28,7 +28,6 @@
 #include <dali-toolkit/internal/text/cursor-helper-functions.h>
 #include <dali-toolkit/internal/text/glyph-metrics-helper.h>
 #include <dali-toolkit/internal/text/layouts/layout-parameters.h>
-#include <dali-toolkit/internal/text/character-set-conversion.h>
 
 namespace Dali
 {
@@ -478,8 +477,7 @@ struct Engine::Impl
     DALI_LOG_INFO( gLogFilter, Debug::Verbose, "<--GetLineLayoutForBox\n" );
   }
 
-  void SetGlyphPositions( const Parameters& layoutParameters,
-                          const GlyphInfo* const glyphsBuffer,
+  void SetGlyphPositions( const GlyphInfo* const glyphsBuffer,
                           Length numberOfGlyphs,
                           float outlineWidth,
                           Vector2* glyphPositionsBuffer )
@@ -496,11 +494,6 @@ struct Engine::Impl
 
     for( GlyphIndex i = 0u; i < numberOfGlyphs; ++i )
     {
-      const CharacterIndex characterIndex = *( layoutParameters.glyphsToCharactersBuffer + i );
-      const Character character = *( layoutParameters.textBuffer + characterIndex );
-      std::string text;
-      Utf32ToUtf8( reinterpret_cast<const uint32_t* const>( &character ), 1, text );
-
       const GlyphInfo& glyph = *( glyphsBuffer + i );
       Vector2& position = *( glyphPositionsBuffer + i );
 
@@ -508,8 +501,6 @@ struct Engine::Impl
       position.y = -glyph.yBearing;
 
       penX += glyph.advance;
-
-      DALI_LOG_ERROR("SetGlyphPositions: text: %s, GlyphIndex: %u, glyph.index: %u, glyph.xBearing: %f, glyph.advance: %f, position.x: %f, penX: %f\n", text.c_str(), i, glyph.index, glyph.xBearing, glyph.advance, position.x, penX);
     }
   }
 
@@ -622,7 +613,7 @@ struct Engine::Impl
         layoutSize.height += ( lineRun->ascender + -lineRun->descender ) + lineRun->lineSpacing;
       }
 
-      SetGlyphPositions( layoutParameters, layoutParameters.glyphsBuffer + lineRun->glyphRun.glyphIndex,
+      SetGlyphPositions( layoutParameters.glyphsBuffer + lineRun->glyphRun.glyphIndex,
                          ellipsisLayout.numberOfGlyphs,
                          layoutParameters.outlineWidth,
                          glyphPositionsBuffer + lineRun->glyphRun.glyphIndex - layoutParameters.startGlyphIndex );
@@ -985,7 +976,7 @@ struct Engine::Impl
         } // whether to add a last line.
 
         // Sets the positions of the glyphs.
-        SetGlyphPositions( layoutParameters, layoutParameters.glyphsBuffer + index,
+        SetGlyphPositions( layoutParameters.glyphsBuffer + index,
                            layout.numberOfGlyphs,
                            layoutParameters.outlineWidth,
                            glyphPositionsBuffer + index - layoutParameters.startGlyphIndex );