From 038f4b4fcf468bec2324c72b31668a9cc39c9d74 Mon Sep 17 00:00:00 2001 From: Jean-Philippe Andre Date: Mon, 13 Jan 2014 15:06:48 +0900 Subject: [PATCH] Evas/cserve2: Reduce error logs on invalid glyph load Change message level from ERR to WRN, when a glyph is not loadable because FT fails to load it or it contains 0 pixel. cserve2 used to complain about invalid glyph 3, on a few fonts --- src/bin/evas/evas_cserve2_fonts.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/bin/evas/evas_cserve2_fonts.c b/src/bin/evas/evas_cserve2_fonts.c index a13630b2d9..006bf27538 100644 --- a/src/bin/evas/evas_cserve2_fonts.c +++ b/src/bin/evas/evas_cserve2_fonts.c @@ -329,6 +329,7 @@ _font_slave_glyph_render(Font_Info *fi, Slave_Msg_Font_Glyphs_Loaded *response, FT_BitmapGlyph bglyph; char *data; int buffer_id = 0; + Eina_Bool valid = EINA_FALSE; void *buf; FT_Get_Glyph(fsi->face->glyph, &glyph); @@ -342,6 +343,7 @@ _font_slave_glyph_render(Font_Info *fi, Slave_Msg_Font_Glyphs_Loaded *response, } glyphsize = bglyph->bitmap.pitch * bglyph->bitmap.rows; + valid = EINA_TRUE; buf = evas_common_font_glyph_compress(bglyph->bitmap.buffer, bglyph->bitmap.num_grays, bglyph->bitmap.pixel_mode, @@ -384,7 +386,10 @@ _font_slave_glyph_render(Font_Info *fi, Slave_Msg_Font_Glyphs_Loaded *response, on_error: // Create invalid entry for this index. There will be an empty slot in // the mempool (usually 8 bytes) because we need the Glyph_Data index entry. - ERR("Could not load glyph %d. Creating empty invalid entry.", idx); + if (valid) + ERR("Failed to load glyph %d. Running out of memory?", idx); + else + WRN("Could not load glyph %d. Creating empty invalid entry.", idx); memset(&response->glyphs[response->nglyphs], 0, sizeof(Slave_Msg_Glyph)); if (buffer_id > 0) cserve2_shared_mempool_buffer_del(response->mempool, buffer_id);